1052466 - [10.10] Update titlebar/toolbar to use transparency on OS X Yosemite

Status: RESOLVED WONTFIX

(Firefox :: Theme, defect)

Description

[:Gijs (he/him)]

The titlebar and toolbar on Yosemite will want to use the blur/transparency that are provided by NSVisualEffectview, and we'll be wanting to use that, in turn, to "fit in" more on OS X 10.10.

Note also that the "fallback" / nontransparent color is different from 10.9, too.

Comment 1

[Wesley Johnston (:wesj)]

This looks nice if you flip it on. And it automatically falls back to the right color for us if you turn off transparency in Accessibility settings. Is there anything special (beyond maybe lwt?) I need to take into account to write a real patch?

There's also a nice dark mode that would be fun to use for private windows. Are the plans for that? https://dl.dropboxusercontent.com/u/72157/Titlebar.png

Comment 2

[Wesley Johnston (:wesj)]

Attachment: Patch

i.e. Is this all you need?

Comment 3

[:Gijs (he/him)]

Comment on attachment 8507484 [details] [diff] [review]
Patch

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

This breaks the titlebar/tabbar vs. navbar border. To fix that, needs to also update the border hsl color added in bug 1065429. .4 opacity looks better to my untrained and tired eyes. Want to update the patch with that? :-)

Markus, does this make sense and/or do you think we should expect dragons somewhere? :-)

Comment 4

[Markus Stange [:mstange]]

Comment on attachment 8507484 [details] [diff] [review]
Patch

We should have eradicated most dragons by now (the sidebar was a good testing ground for that), but I'd prefer if we don't land this until font subpixel antialiasing is fixed. I only need one more review from jrmuizel in bug 1055622 before I can land those patches.

Comment 5

[:Gijs (he/him)]

(In reply to Markus Stange [:mstange] from comment #4)
> Comment on attachment 8507484 [details] [diff] [review]
> Patch
> 
> We should have eradicated most dragons by now (the sidebar was a good
> testing ground for that), but I'd prefer if we don't land this until font
> subpixel antialiasing is fixed. I only need one more review from jrmuizel in
> bug 1055622 before I can land those patches.

This can land now, right?

Comment 6

[Markus Stange [:mstange]]

Correct!

Comment 7

[:Gijs (he/him)]

remote:   https://hg.mozilla.org/integration/fx-team/rev/6bb4109d3f5e

Comment 8

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/6bb4109d3f5e

Comment 9

[José Jeria]

The way the transparency was added looks more how Windows 7 used transparency compared to Yosemite...

Comment 10

[Markus Stange [:mstange]]

After trying to get used to this for the past few weeks, I still don't really like it. The original idea was to use both kinds of vibrancy (behind-window and in-window) in the Firefox UI: in-window vibrancy for the toolbar and the active tab, and behind-window vibrancy for the tab bar background. This would have shown that this style was a conscious decision on our side and not caused by platform limitations. But now that we've decided to drop in-window vibrancy (bug 1050804), we only have behind-window vibrancy, and we're using it in a place where it doesn't belong. Can we go back to the opaque toolbar style again? We'll still be using behind-window vibrancy for sidebars, which are the right places to use the effect.

Comment 11

[(Currently slow to respond) Philipp Sackl [:phlsa] (Firefox UX) please use needinfo]

(In reply to Markus Stange [:mstange] from comment #10)
> After trying to get used to this for the past few weeks, I still don't
> really like it. The original idea was to use both kinds of vibrancy
> (behind-window and in-window) in the Firefox UI: in-window vibrancy for the
> toolbar and the active tab, and behind-window vibrancy for the tab bar
> background. This would have shown that this style was a conscious decision
> on our side and not caused by platform limitations. But now that we've
> decided to drop in-window vibrancy (bug 1050804), we only have behind-window
> vibrancy, and we're using it in a place where it doesn't belong. Can we go
> back to the opaque toolbar style again? We'll still be using behind-window
> vibrancy for sidebars, which are the right places to use the effect.

Well, the kind of toolbar/titlebar combo that Firefox uses isn't used by any other OS X app that I'm aware of. I think that what we are doing right now is in the spirit of the Yosemite design, even though it doesn't follow platform conventions to the letter.

But since this is mostly a visual design decision, I'll defer to Michael…

Comment 12

[Markus Stange [:mstange]]

This change has now been reverted in bug 1161565. No out-of-place vibrancy any more! :)