Closed Bug 1926394 Opened 1 year ago Closed 1 year ago

No animation when dragging a tab over other tabs in a tab group

Categories

(Firefox :: Tabbed Browser, defect, P3)

defect
Points:
1

Tracking

()

VERIFIED FIXED
133 Branch
Tracking Status
firefox133 --- verified
firefox134 --- verified

People

(Reporter: amylee, Assigned: dao)

References

(Blocks 1 open bug)

Details

(Whiteboard: [fidefe-tabgrps-dnd][UX-tabgrps-review])

Attachments

(2 files)

Attached video no animation.mov

Steps to reproduce

Create a tab group with 1+ tabs inside
Open several new tabs outside the tab group
Drag a tab into the tab group

Expected
There is a sliding animation when you drag a tab inside a tab group (same as when you move a tab when not in a group)

Actual
No sliding animation when moving a tab inside a group

See screen recording

Blocks: 190710
Severity: -- → S4
Priority: -- → P3
Summary: No animation when moving a tab inside a tab group → No animation when dragging a tab over other tabs in a tab group
Whiteboard: [fidefe-tabgrps-tabbrowser] [UX-tabgrps-review] → [fidefe-tabgrps-dnd][UX-tabgrps-review]
Points: --- → 1
Blocks: 1907101
No longer blocks: 190710
Assignee: nobody → dao+bmo
Status: NEW → ASSIGNED
Pushed by dgottwald@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/79c92abcd174 Update [movingtab] selectors to account for grouped tabs. r=dwalker,tabbrowser-reviewers

Tagging Amy to review this in the next Nightly build after this bug gets resolved.

Flags: needinfo?(amlee)

Backed out for causing mochitests failures in browser_preserveTabPipIconOverlay.js.

Also please check these mochitests failures, these mochitests failures and these mochitests failures.

Flags: needinfo?(dao+bmo)
Pushed by dgottwald@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/9ae61137c8c5 Update [movingtab] selectors to account for grouped tabs. r=jswinarton,dwalker,tabbrowser-reviewers
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → 133 Branch
Flags: qe-verify+
Flags: needinfo?(dao+bmo)

Looks good. Thanks!

Flags: needinfo?(amlee)

I was able to reproduce the issue on Win10x64 using Firefox build 133.0a1 (2024-10-22).
Verified as fixed on Win10x64, macOS 13.6 ARM and Ubuntu 24.04 using Firefox build 134.0a1 and 133.0b9.

Status: RESOLVED → VERIFIED
Flags: qe-verify+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: