Closed Bug 1949401 Opened 6 months ago Closed 1 month ago

Active Tabs should remain active for collapsed Tab groups

Categories

(Firefox :: Tabbed Browser, enhancement, P2)

Desktop
Unspecified
enhancement
Points:
3

Tracking

()

VERIFIED FIXED
142 Branch
Tracking Status
firefox-esr128 --- disabled
firefox-esr140 --- wontfix
firefox135 --- disabled
firefox136 --- disabled
firefox137 --- wontfix
firefox140 --- wontfix
firefox141 --- wontfix
firefox142 --- verified

People

(Reporter: rdoghi, Assigned: dwalker)

References

(Depends on 1 open bug, Blocks 3 open bugs, Regressed 1 open bug)

Details

(Whiteboard: [fidefe-tabgrps-tabbrowser])

Attachments

(2 files)

Found in

  • 137.0a1 (2025-02-20)

Affected versions

  • 137.0a1 (2025-02-20)

Affected platforms

  • all

Preconditions

  • NA

Steps to reproduce

  1. Create a Tab group.
  2. Select any Tab from the Tab group.
  3. Collapse the Tab group.

Expected result

  • The Active tab should still be displayed.
    An indicator showing that the Active Tab is inside that Tab group is also needed for when we have a lot of collapsed tab groups.

Actual result

  • The Moment we collapse a tab group it will jump to either a new tab or a different available tab.
    Some users would collapse the tab group instead of scrolling in order to see more tabs, but they wouldnt want to switch the active tab from the page they were on. Jumping to a different active tab without users input might not be ideal when they simply collapse the Tab group.

Regression range
Not a Regression

Blocks: 1907099
Severity: -- → N/A
Points: --- → 3
Priority: -- → P2
Whiteboard: [fidefe-tabgrps-tabbrowser]
Blocks: 1953207
Assignee: nobody → dwalker

Notes about what we need to change/preserve when doing this work:

  1. Add a third state for tab groups (“semi-collapsed”) to control styling (horizontal and vertical)
  2. Keep track of group tab count to display overflow indicator (horizontal and vertical)
  3. Ensure active tab is not considered hidden when in a collapsed group
  4. Ensure active tab is in aria focus order when in a collapsed group
  5. List all tabs menu should display the active tab even if it’s in a collapsed group
  6. Ensure a semi-collapsed group displays correctly when restoring a session
  7. Ensure we blur correctly when closing the active tab in a collapsed group
  8. Ensure we transition to open/collapsed/semicollapsed correctly when closing the active tab in a collapsed group
Blocks: 1971232
Depends on: 1972601
Attachment #9495284 - Attachment description: WIP: Bug 1949401 - maintain visibility of selected tab when group is collapsed. r?sthompson,jswinarton → Bug 1949401 - maintain visibility of selected tab when group is collapsed. r?sthompson,jswinarton
Blocks: 1973286
Depends on: 1973519
No longer depends on: 1973519
Pushed by dwalker@mozilla.com: https://github.com/mozilla-firefox/firefox/commit/ae9aac93b8d8 https://hg.mozilla.org/integration/autoland/rev/060a6fba17f5 maintain visibility of selected tab when group is collapsed. r=sthompson,desktop-theme-reviewers,tabbrowser-reviewers,fxview-reviewers,urlbar-reviewers,dao
Status: NEW → RESOLVED
Closed: 1 month ago
Resolution: --- → FIXED
Target Milestone: --- → 142 Branch

This is so cool, how it also shows the number of tabs in that group.. This issue is verified as fixed in our latest Nightly build 142.0a1 (2025-07-03). Love this upgrade to Tab groups.

Attached video DragTab.mp4

Can someone take a look at this issue, I think this fix is causing it.
When we try to move tabs with the Vertical Sidebar the Tabs are no longer displayed underneath the actual mouse cursor and I think it happens because of the extra +nr of tabs from the collapsed groups, I also noticed that not all of them will show that number, it usually enables if we collapse a group with an active tab in it. Please take a look at the video.

Let me know if I should log a separate issue for this, also what is the correct behavior should the extra +tab nr show up for each collapsed group or not ?

Flags: needinfo?(dwalker)
QA Whiteboard: [qa-ver-needed-c142/b141]
Blocks: 1975486

(In reply to Rares Doghi, Desktop QA from comment #6)

Created attachment 9498329 [details]
DragTab.mp4

Can someone take a look at this issue, I think this fix is causing it.
When we try to move tabs with the Vertical Sidebar the Tabs are no longer displayed underneath the actual mouse cursor and I think it happens because of the extra +nr of tabs from the collapsed groups, I also noticed that not all of them will show that number, it usually enables if we collapse a group with an active tab in it. Please take a look at the video.

Let me know if I should log a separate issue for this, also what is the correct behavior should the extra +tab nr show up for each collapsed group or not ?

I just filed bug 1975486 to address this. The overflow counter should not display at all if the group doesn't contain the active tab, but there is a bug that occurs when switching between horizontal and vertical tabs mode.

Flags: needinfo?(dwalker)

Updating the main status flag.

Status: RESOLVED → VERIFIED
QA Whiteboard: [qa-ver-needed-c142/b141] → [qa-ver-done-c142/b141]
See Also: → 1951855
Regressions: 1979067
Depends on: 1979068
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: