Closed Bug 714281 Opened 13 years ago Closed 13 years ago

Show the all tabs button only when the tab strip overflows

Categories

(Firefox :: Tabbed Browser, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
Firefox 12

People

(Reporter: dao, Assigned: dao)

References

(Blocks 1 open bug)

Details

(Keywords: ux-minimalism)

Attachments

(1 file, 1 obsolete file)

bug 626854 spin-off
Blocks: 607226
Attached patch patch (obsolete) (deleted) — Splinter Review
Frank, I don't see how this could cause the scrollbox to "continously toggle between overflow and underflow." Is this still a concern here?
Attachment #585123 - Flags: feedback?(fryn)
Comment on attachment 585123 [details] [diff] [review] patch (In reply to Dão Gottwald [:dao] from comment #1) > Frank, I don't see how this could cause the scrollbox to "continously toggle > between overflow and underflow." Is this still a concern here? I cannot reproduce the problem with this patch. This patch regresses the feature of bug 465086. We need a bit more to address that, e.g. something like replacing the _usingClosingTabsSpacer *property* with a locked attribute and adding the following to the stylesheet: #tabbrowser-tabs[locked]:not([overflow="true"]) ~ #alltabs-button { + visibility: hidden; +} `visibility: hidden;` conveniently also blocks pointer events. Thanks for taking on this bug.
Attachment #585123 - Flags: feedback?(fryn) → feedback-
That should've been: #tabbrowser-tabs[locked]:not([overflow="true"]) ~ #alltabs-button { visibility: hidden; } Ignore the pluses. (Copy-paste fail.)
Attached patch patch v2 (deleted) — Splinter Review
"locked" seems too vague, so I didn't use that.
Assignee: nobody → dao
Attachment #585123 - Attachment is obsolete: true
Status: NEW → ASSIGNED
Attachment #585262 - Flags: ui-review?(limi)
Attachment #585262 - Flags: review?
Attachment #585262 - Flags: feedback?(fryn)
Blocks: 714594
Attachment #585262 - Flags: review? → review?(mak77)
Comment on attachment 585262 [details] [diff] [review] patch v2 Looks good!
Attachment #585262 - Flags: feedback?(fryn) → feedback+
IIRC, the panorama button is hidden till it's used first time, and the alltabs menuitem for it is the only accessible (in the sense of visible) hook to open Panorama. Do we still do that hiding-by-default? If so, the question is whether we consider Panorama a useful feature to be discoverable when tabs don't overflow.
(In reply to Marco Bonardo [:mak] from comment #6) > IIRC, the panorama button is hidden till it's used first time, and the > alltabs menuitem for it is the only accessible (in the sense of visible) > hook to open Panorama. > Do we still do that hiding-by-default? Yes. > If so, the question is whether we > consider Panorama a useful feature to be discoverable when tabs don't > overflow. See bug 626854 comment 14... Panorama is mostly useful for users with so many tabs that they become hard to handle on a single tab strip. "Many tabs" more or less implies overflow, so I think this is still covered well enough.
Comment on attachment 585262 [details] [diff] [review] patch v2 Review of attachment 585262 [details] [diff] [review]: ----------------------------------------------------------------- ::: browser/base/content/browser.css @@ +22,5 @@ > } > > +#tabbrowser-tabs:not([overflow="true"])[using-closing-tabs-spacer] ~ #alltabs-button { > + visibility: hidden; > +} So, I assume the hidden override is added to avoid the flicker. A brief comment above it may clarify the reason to future generations
Attachment #585262 - Flags: review?(mak77) → review+
Comment on attachment 585262 [details] [diff] [review] patch v2 Yes, please. :)
Attachment #585262 - Flags: ui-review?(limi) → ui-review+
(In reply to Marco Bonardo [:mak] from comment #8) > So, I assume the hidden override is added to avoid the flicker. A brief > comment above it may clarify the reason to future generations It's not to avoid flicker. It's to maintain the correct amount of temporary space needed for the next tab's close button to be positioned under the cursor, so we don't regress bug 465086. Still, a comment would indeed clarify this. I'd like to push this now, since I have dependent changes that need unbitrotting, so I'll add a comment when doing that. Thanks for the review. :)
Target Milestone: --- → Firefox 12
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Errrm, where is "Tabs from other computers" accessible now? I can't find it on Windows now (without unhiding the old menu bar).
(In reply to Thomas Stache from comment #13) > Errrm, where is "Tabs from other computers" accessible now? I can't find it > on Windows now (without unhiding the old menu bar). filed bug 716271
Depends on: 716271
Thanks Dão, I was sure this was only an oversight...
Sorry to be a UI complainer, but is there a way I can override this for myself? I have a user style sheet that prevents tab strip overflow and has no minimum tab width. When the tabs get a bit small, I like being able to use the menu to find a tab. Thanks!
Depends on: 702005
Blocks: 702005
No longer depends on: 702005
(In reply to Cameron McCormack (:heycam) from comment #16) > Sorry to be a UI complainer, but is there a way I can override this for > myself? I have a user style sheet that prevents tab strip overflow and has > no minimum tab width. When the tabs get a bit small, I like being able to > use the menu to find a tab. Thanks! I shrink tabs to favicons and use(d) the dropdown to find articles if I want to jump to a specific tab; not even sure why the change is needed, just move the icon into the customise window… You can get it back by setting this in your userChrome/Stylish #tabbrowser-tabs ~ #alltabs-button { visibility: visible #important; }
Personally I wish it was left or at minimum placed in the "Customize Toolbar" selection like the "Tab Groups" is. I preferred List All Tabs since it gave me quick access to both the "Tabs from other computers" and "Tab Groups" but also allowing for view of a list.
Note that this means that the panorama button is not available unless you add it manually in the customize window, or if you know the keyboard shortcut. Contradicts <http://support.mozilla.org/en-US/kb/what-are-tab-groups>
(In reply to John Drinkwater (:beta) from comment #17) > > You can get it back by setting this in your userChrome/Stylish > #tabbrowser-tabs ~ #alltabs-button { visibility: visible #important; } #tabbrowser-tabs ~ #alltabs-button { visibility: visible !important; }
(In reply to al_9x from comment #20) > (In reply to John Drinkwater (:beta) from comment #17) > > #tabbrowser-tabs ~ #alltabs-button { visibility: visible #important; } > > #tabbrowser-tabs ~ #alltabs-button { visibility: visible !important; } Oops, my bad. Honest typo I swear :)
Well, for me this problem isn't fixed or resolved! I'm using Firefox on an iMac with OS X 10.5.8. Until a week or two ago, when I clicked on the "Windows" menu at the top of my screen, it would show me a list of all the tabs I had open, but this handy helper suddenly stopped. Now, when I click on "Windows" it only shows me the open windows. I just upgraded to Firefox 13.0.1, but the list of all open tabs is still not working. Is this the Mac version of the missing "List All Tabs" button? I tried using the "Permanent List-all-tabs Button" add-on, but it didn't do anything, presumably because I'm using a Mac. Is there any way I can get back my handy list of all open tabs??
Depends on: 928011
Depends on: 1153751
Depends on: 1406255
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: