Closed Bug 1943885 Opened 1 year ago Closed 1 year ago

Pinning first tab should ungroup it

Categories

(Firefox :: Tabbed Browser, defect, P1)

defect
Points:
1

Tracking

()

RESOLVED FIXED
136 Branch
Tracking Status
firefox136 --- fixed

People

(Reporter: emilio, Assigned: dao)

References

(Blocks 1 open bug)

Details

(Whiteboard: [fidefe-tabgrps-tabbrowser])

Attachments

(3 files)

Attached image Screenshot 1

When you have a tab group and pin one of its tabs, behavior is a bit erratic:

  • The first pinned tab seems to work (see attached screenshot).
  • But as soon as you pin another tab from another group, then that tab gets detached from its group.
  • If you add many tabs, so that the tab list scrolls, then the pinned tab gets completely inaccessible.

I'd expect either:

  • Pinning a tab always "ungroups" it. A bit unfortunate since it might be useful to have a tab group in the pinned tab area, but maybe ok.
  • Pinning a tab pins the whole group.

But I haven't thought too hard about this. In any case making the tab inaccessible seems clearly a bug.

Note how the pinned tab is gone.

Blocks: tab-groups
Blocks: 1907099
No longer blocks: tab-groups
Severity: -- → S3
Points: --- → 1
Priority: -- → P2
Summary: Tab groups interact poorly with pinned tabs → Pinning a tab should ungroup it
Whiteboard: [fidefe-tabgrps-tabbrowser]
Priority: P2 → P1

I disagree: I always wanted to be able to pin some (not all) tabs in groups.

Depends on: 1927994
Summary: Pinning a tab should ungroup it → Pinning first tab should ungroup it
Assignee: nobody → dao+bmo
Status: NEW → ASSIGNED
Pushed by dgottwald@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/62e03ac130ac Pinning first tab should ungroup it. r=dwalker,tabbrowser-reviewers
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → 136 Branch

#c2

David, likewise: I literally have issues.chromium.org/issues/40937562 and 40937561 filed with Chromium for this. Would an FR to re-add this functionality in a non-buggy manner be acceptable?

Flags: needinfo?(dao+bmo)

(In reply to Mr. Beedell, Roke Julian Lockhart from comment #6)

#c2

David, likewise: I literally have issues.chromium.org/issues/40937562 and 40937561 filed with Chromium for this. Would an FR to re-add this functionality in a non-buggy manner be acceptable?

We have a few more edge cases to consider e.g. related to tab overflow (which Chrome doesn't have). So I don't know how realistic it is to come up with a solution here, and it's not clear to me what that solution might be, but with that being said, sure, we would consider a non-buggy implementation.

Flags: needinfo?(dao+bmo)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: