Closed Bug 932344 Opened 7 years ago Closed 7 years ago

nav-bar is missing top border in private browsing windows on UX

Categories

(Firefox :: Theme, defect)

x86_64
macOS
defect
Not set
normal

Tracking

()

RESOLVED FIXED
Firefox 28

People

(Reporter: mconley, Assigned: Gijs)

References

Details

(Whiteboard: [Australis:P2][Australis:M9])

Attachments

(1 file)

mart3ll reported this to me on IRC.

STR:

1) Open a PB window on OS X using the UX branch
2) Notice the missing border between the top of the nav-bar and the titlebar (see http://cl.ly/image/21363i0s2l0C)
Oh boo. I guess in this case we need to draw the border ourselves still?
(In reply to :Gijs Kruitbosch from comment #1)
> Oh boo. I guess in this case we need to draw the border ourselves still?

Yeah, for private browsing windows - since we're not using the native titlebar gradient (which comes with its own bottom border).
Couldn't we use the native titlebar gradient in private browsing windows, too? I guess we're not using it because the private browsing marker in the top right corner is a background image on the window, and the title bar would cover it... can we make it a ::before on #titlebar, maybe?
(In reply to Markus Stange [:mstange] from comment #3)
> Couldn't we use the native titlebar gradient in private browsing windows,
> too? I guess we're not using it because the private browsing marker in the
> top right corner is a background image on the window, and the title bar
> would cover it... can we make it a ::before on #titlebar, maybe?

We could, but that sounds like it'll introduce more bugs. I think we want to keep a custom design anyway, because in the future we'll have a completely different one (darker than the current one). Can we just take the changeset that removed it and reapply it in the private browsing case only? Sounds like it shouldn't be too bad.
Whiteboard: [Australis:P4][Australis:M?] → [Australis:P2][Australis:M?]
It's also missing in customize mode.
This should do it... :-)
Attachment #826189 - Flags: review?(mnoorenberghe+bmo)
Assignee: nobody → gijskruitbosch+bugs
Status: NEW → ASSIGNED
Attachment #826189 - Flags: review?(mnoorenberghe+bmo) → review+
Comment on attachment 826189 [details] [diff] [review]
draw border for customize mode and fullscreen,

Please make the commit message more detailed to indicate which border you are talking about.
https://hg.mozilla.org/projects/ux/rev/42461ce3dcd0
Updated commit msg: draw tabstoolbar bottom border ourselves during customize mode, private browsing mode and fullscreen
Whiteboard: [Australis:P2][Australis:M?] → [Australis:P2][Australis:M9][fixed-in-ux]
https://hg.mozilla.org/mozilla-central/rev/42461ce3dcd0
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Whiteboard: [Australis:P2][Australis:M9][fixed-in-ux] → [Australis:P2][Australis:M9]
Target Milestone: --- → Firefox 28
You need to log in before you can comment on or make changes to this bug.