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)
Firefox
Tabbed Browser
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)
(deleted),
patch
|
mak
:
review+
limi
:
ui-review+
fryn
:
feedback+
|
Details | Diff | Splinter Review |
bug 626854 spin-off
Assignee | ||
Comment 1•13 years ago
|
||
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 2•13 years ago
|
||
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-
Comment 3•13 years ago
|
||
That should've been:
#tabbrowser-tabs[locked]:not([overflow="true"]) ~ #alltabs-button {
visibility: hidden;
}
Ignore the pluses. (Copy-paste fail.)
Assignee | ||
Comment 4•13 years ago
|
||
"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)
Assignee | ||
Updated•13 years ago
|
Attachment #585262 -
Flags: review? → review?(mak77)
Comment 5•13 years ago
|
||
Comment on attachment 585262 [details] [diff] [review]
patch v2
Looks good!
Attachment #585262 -
Flags: feedback?(fryn) → feedback+
Comment 6•13 years ago
|
||
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.
Assignee | ||
Comment 7•13 years ago
|
||
(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 8•13 years ago
|
||
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 9•13 years ago
|
||
Comment on attachment 585262 [details] [diff] [review]
patch v2
Yes, please. :)
Attachment #585262 -
Flags: ui-review?(limi) → ui-review+
Comment 10•13 years ago
|
||
(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. :)
Assignee | ||
Comment 11•13 years ago
|
||
Target Milestone: --- → Firefox 12
Comment 12•13 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Comment 13•13 years ago
|
||
Errrm, where is "Tabs from other computers" accessible now? I can't find it on Windows now (without unhiding the old menu bar).
Assignee | ||
Comment 14•13 years ago
|
||
(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
Comment 15•13 years ago
|
||
Thanks Dão, I was sure this was only an oversight...
Comment 16•13 years ago
|
||
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!
Assignee | ||
Updated•13 years ago
|
Comment 17•13 years ago
|
||
(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; }
Comment 18•13 years ago
|
||
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.
Comment 19•13 years ago
|
||
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>
Comment 20•13 years ago
|
||
(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; }
Comment 21•13 years ago
|
||
(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 :)
Comment 22•12 years ago
|
||
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??
Assignee | ||
Updated•6 years ago
|
You need to log in
before you can comment on or make changes to this bug.
Description
•