Closed Bug 1926382 Opened 1 month ago Closed 16 days ago

Group label covers tab when dragging a tab

Categories

(Firefox :: Tabbed Browser, defect, P1)

defect
Points:
3

Tracking

()

RESOLVED FIXED
134 Branch
Tracking Status
firefox134 --- fixed

People

(Reporter: amylee, Assigned: dao)

References

(Blocks 2 open bugs)

Details

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

Attachments

(2 files, 2 obsolete files)

Attached video label covers tab.mov

Steps to reproduce:

  1. Create 2 tab groups with 2+ tabs inside each
  2. Drag a tab from one group to the other

Expected:
Labels don't overlap with tabs
Tab group hairline doesn't follow the tab once it's moved out of the group

Actual:

Tab group label overlaps with tab when moving a tab between groups
Tab group hairline follows tab when moved out of the group

See screen recording for reference

Blocks: 1908437, 1907101
Severity: -- → S3
Points: --- → 2
Priority: -- → P2
Whiteboard: [fidefe-tabgrps-tabbrowser] [UX-tabgrps-review] → [fidefe-tabgrps-dnd][UX-tabgrps-review]
Summary: Group label covers tab when moving a tab → Group label covers tab when dragging a tab
Priority: P2 → P1
Whiteboard: [fidefe-tabgrps-dnd][UX-tabgrps-review] → [fidefe-tabgrps-dnd][UX-tabgrps-review] [fidefe-tabgrps-dnd]
Whiteboard: [fidefe-tabgrps-dnd][UX-tabgrps-review] [fidefe-tabgrps-dnd] → [fidefe-tabgrps-dnd][UX-tabgrps-review]
Points: 2 → 3
No longer blocks: 1908437
Assignee: nobody → dao+bmo
Status: NEW → ASSIGNED
Pushed by dgottwald@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/4424e72e5872 Shift group labels along with the group's first tab. r=sthompson,tabbrowser-reviewers

Comment on attachment 9437620 [details]
WIP: Bug 1926382 - hacky way to shift tab group labels

Revision D228932 was moved to bug 1908439. Setting attachment 9437620 [details] to obsolete.

Attachment #9437620 - Attachment is obsolete: true

Comment on attachment 9437621 [details]
WIP: Bug 1926382 - refactor tab dragdrop to item dragdrop

Revision D228933 was moved to bug 1908439. Setting attachment 9437621 [details] to obsolete.

Attachment #9437621 - Attachment is obsolete: true
Status: ASSIGNED → RESOLVED
Closed: 16 days ago
Resolution: --- → FIXED
Target Milestone: --- → 134 Branch
Blocks: 1932489
Flags: qe-verify+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: