972140 - Australis pinned tabs break toolbar interface at small window sizes

Status: VERIFIED FIXED

(Firefox :: Toolbars and Customization, defect)

Description

[Jennifer Morrow [:Boriss] (UX)]

Attachment: Screenshot: Small window with pinned tabs showing interface errors

When multiple pinned tabs are present in an OSX Australis window, visual clipping and artifacts appear in the toolbar UI.  

There's three separate points of interface fail (see attached):

a. Misaligned pinned tab
b. Overlap of fullsize and new tab button
C. Misaligned favicons

Steps to reproduce:

1. On OSX, create at least 5 pinned tabs in one window
2. Reduce the window size

Comment 1

[Dão Gottwald [:dao]]

Is this even an Australis regression?

Comment 2

[:Gijs (he/him)]

(In reply to Dão Gottwald [:dao] from comment #1)
> Is this even an Australis regression?

At least on OS X it is, apart from the weird overflow UI which all overlaps, even on holly. Everything else is tabsintitlebar related. Considering we used to only do this in maximized mode on Windows, it would have been a lot harder to hit there. Don't know if (parts of) this bug even reproduce(s) on Windows in the first place.

Comment 3

[Justin Dolske [:Dolske]]

Attachment: Button mess

I can't reproduce the  tab/icon problem in the screenshot ("a" and "c").

But I do get the New Tab and All Tabs buttons sliding over the next/previous tab overflow arrows, and if the window gets a bit smaller still the Fullscreen window button with slide over that mess as well.

Comment 4

[Justin Dolske [:Dolske]]

Calling this P3- because while breaking primary UI is terrible, it's OS X only (?) and I'd tend to think that people will keep windows with pinned tabs fairly large (as opposed to a problem with popup windows, which tend to be small).

Comment 5

[:Gijs (he/him)]

This reproduces on WinXP in Australis. Fixing (a) and (c) is not hard, patch coming up presto. I think (b) is pre-Australis, and once we fix (a) and (c) that can be lower priority.

Comment 6

[:Gijs (he/him)]

Attachment: fix tabbrowser-tabs min-height for Australis so it doesn't get squashed when the non-abs-pos bits have 0 width,

Comment 7

[Mike Conley (:mconley) (:⚙️)]

Comment on attachment 8384590 [details] [diff] [review]
fix tabbrowser-tabs min-height for Australis so it doesn't get squashed when the non-abs-pos bits have 0 width,

Review of attachment 8384590 [details] [diff] [review]:
-----------------------------------------------------------------

Makes sense to me. Thanks Gijs!

Comment 8

[:Gijs (he/him)]

remote:   https://hg.mozilla.org/integration/fx-team/rev/9a9f7dd350aa

Comment 9

[:Gijs (he/him)]

Filed bug 978835 for (b), so we can close this when the fix merges, uplifts etc.

Comment 10

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/9a9f7dd350aa

Comment 11

[:Gijs (he/him)]

Comment on attachment 8384590 [details] [diff] [review]
fix tabbrowser-tabs min-height for Australis so it doesn't get squashed when the non-abs-pos bits have 0 width,

[Approval Request Comment]
Bug caused by (feature/regressing bug #): Australis
User impact if declined: window goes wonky when small and only a few pinned tabs are left
Testing completed (on m-c, etc.): on m-c, local
Risk to taking this patch (and alternatives if risky): very low - localized change that changes tabstrip height to match tabs
String or IDL/UUID changes made by this patch: none

Comment 12

[:Gijs (he/him)]

https://hg.mozilla.org/releases/mozilla-aurora/rev/56fdd858c8ce

Comment 13

[Cornel Ionce [:noni] [Hubs QA]]

Verified that first issue is no longer reproducing on Windows 7 64 bit, Mac OS X 10.9 and Ubuntu 12.04 using:
- latest Nightly (build ID: 20140313030202).
- latest Aurora (build ID: 20140313004000).

Marking verified as bug 978835 was filled for the remaining issue.