Skip to content

Fix (UI): Extend input draggability fix to all browsers#9100

Open
ufuksarp wants to merge 4 commits intoinvoke-ai:mainfrom
ufuksarp:fix-extend-dnd-input-fix
Open

Fix (UI): Extend input draggability fix to all browsers#9100
ufuksarp wants to merge 4 commits intoinvoke-ai:mainfrom
ufuksarp:fix-extend-dnd-input-fix

Conversation

@ufuksarp
Copy link
Copy Markdown
Contributor

Summary

This PR extends the existing input/textarea draggability fix to all browsers, which previously only covered Firefox.

Changes:

  • Removed the Firefox user agent guard so the mouseover/mouseout draggable toggle runs in all browsers.
  • Renamed firefoxDndFix to dndInputFix. All 6 callers updated.
  • Edited relevant comments.
  • Removed the now-unused noop import.

Related Issues / Discussions

None.

QA Instructions

In a Chromium-based browser:

  1. Open the canvas and add a raster layer.
  2. From the layer's context menu, add Adjustments.
  3. In the Adjustments panel, try to click and drag over the value in number input boxes.
  4. Before this PR the layer row starts being dragged for reordering. After this PR the text inside the input gets selected normally.
  5. Confirm dragging the layer row itself still works as expected when the drag starts from a non-input area (header, title, etc).
  6. Also retest in Firefox to confirm no regression of the original fix.

Merge Plan

Standard merge.

Checklist

  • The PR has a short but descriptive title, suitable for a changelog
  • Tests added / updated (if applicable)
  • ❗Changes to a redux slice have a corresponding migration
  • Documentation added / updated (if applicable)
  • Updated What's New copy (if doing a release after this PR)

@github-actions github-actions Bot added the frontend PRs that change frontend files label Apr 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

frontend PRs that change frontend files

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants