Closed Bug 1939518 Opened 1 year ago Closed 1 year ago

Hairline doesn't move in vertical tab groups when moving tabs

Categories

(Firefox :: Tabbed Browser, defect, P1)

Desktop
All
defect
Points:
3

Tracking

()

VERIFIED FIXED
138 Branch
Tracking Status
firefox138 --- verified
firefox139 --- verified

People

(Reporter: amylee, Assigned: dao)

References

(Blocks 1 open bug)

Details

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

Attachments

(2 files)

Attached video Vertica tab group.mov —

Steps to reproduce [can add screenshots or video recordings for reference]:

  1. Enable vertical tabs
  2. Create a tab group
  3. Drag an ungrouped tab that is above or below the tab group

Expected behavior [What should have happened?]:
Tab group repositions as you drag a tab past the opened tab group

Actual behavior [What actually happened?]:

Tab label and tabs shift but hairline is fixed in place

Blocks: 1907534
Points: --- → 3
Priority: -- → P3
Whiteboard: [fidefe-tabgrps-tabbrowser] [UX-tabgrps-review] → [fidefe-tabgrps-vertical] [UX-tabgrps-review]
Severity: -- → S3
See Also: → 1939537
Priority: P3 → P1
Assignee: nobody → dao+bmo
Status: NEW → ASSIGNED
Depends on: 1937860
Blocks: 1939537
Blocks: 1937860
No longer depends on: 1937860
Blocks: 1956314
Regressions: 1956314
Pushed by sthompson@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/a741f9b130df Reimplement vertical tab group line to be more consistent with horizontal tabs, and move container tab line to the end of the tab. r=sthompson,tabbrowser-reviewers
Duplicate of this bug: 1939537
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → 138 Branch
Regressions: 1957409
Regressions: 1957453
Flags: qe-verify+

Hi Dao! There are three regressions for this bug. Should we back this out of beta for 138 or are those bugs going to be prioritized so that we can uplift them? Please advise.

Flags: needinfo?(dao+bmo)

It appears that the regressions have been fixed and verified and the current report was fixed as well so the NI can be removed.

Furthermore, I can reproduce this issue in Nightly v138.0a1 from 2025-03-27 and I can confirm the fix in Nightrly v138.0a1 from 2025-03-29, Nightly v139.0a1 from 2024-04-24 and Beta v138.0 (RC). The "hairline" (the colored line/segment that determines which tabs are included in the group) is now dynamic when a user drags an ungrouped tab over the grouped tabs. Verified in Windows 10, MacOS 11 and Ubuntu 22.

Status: RESOLVED → VERIFIED
Has STR: --- → yes
Flags: qe-verify+
Flags: needinfo?(dao+bmo)
OS: Unspecified → All
Hardware: Unspecified → Desktop
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: