This is fallout from bug 973694 - we're temporarily showing the old private browsing mode flat colour during the customize mode transition. Looks like ass. Patch coming up.
Should have caught this the first time around. Basically, we set the -moz-appearance of the #titlebar to none in order to make so that we can apply a nice textured background to it (the grid) in customize mode. We only need to do that on customize-entered, since this is when the grid is applied. Before then, we should keep the titlebar around. We can discount LWT's, because during the transition, LWT's have been unapplied. Requesting review from either Gijs or MattN. I'll let you two duke it out.
This doesn't break anything functionally speaking, but it certainly detracts from the smoothness of the transition, which I think we'd like to preserve if at all possible.
Comment on attachment 8406382 [details] [diff] [review] Patch v1 That's an empty patch. *sigh*, qref. Brb.
Here we are.
Comment on attachment 8406389 [details] [diff] [review] Patch v1.1 Review of attachment 8406389 [details] [diff] [review]: ----------------------------------------------------------------- rs=me, but let's either trypush or monitor fx-team/m-c carefully as we land this to check CART does OK.
Since we're essentially going back to the same behaviour as before, I'm not too too worried about CART, but I'll keep my eye on it. Thanks! remote: https://hg.mozilla.org/integration/fx-team/rev/bba21dfbd381
Whiteboard: [Australis:P3+] → [Australis:P3+][fixed-in-fx-team]
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Whiteboard: [Australis:P3+][fixed-in-fx-team] → [Australis:P3+]
Target Milestone: --- → Firefox 31
Comment on attachment 8406389 [details] [diff] [review] Patch v1.1 [Approval Request Comment] Bug caused by (feature/regressing bug #): Fallout from a badly selected attribute in a CSS rule from bug 973694. User impact if declined: A rather unpleasant flash of a flat background colour on the titlebar during the middle of the customize mode transition. Testing completed (on m-c, etc.): Local testing. Risk to taking this patch (and alternatives if risky): Extremely low - CSS only, and very much isolated to just the customize mode transition. String or IDL/UUID changes made by this patch: None.
Comment on attachment 8406389 [details] [diff] [review] Patch v1.1 Approving because it is a low risk and a tiny patch (and also because Australis matters)
5 years ago
This was fixed for normal mode, but not for fullscreen mode. Verified as fixed on Mac OS X 10.9.2 using Firefox 29.0b9 and the 04/17 Aurora and Nightly. Filed bug 998275 for fullscreen.
You need to log in before you can comment on or make changes to this bug.