1386721 - [Photon] Add vibrancy to tabstrip in Light theme

Status: VERIFIED FIXED

(Firefox :: Theme, enhancement, P1)

Comment 2

[(inactive) Nihanth Subramanya [:nhnt11]]

Attachment: Bug 1386721 - Add vibrancy to tabstrip in light theme.

Review commit: https://reviewboard.mozilla.org/r/193384/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/193384/

Comment 3

[Dão Gottwald [:dao]]

Comment on attachment 8922335 [details]
Bug 1386721 - Add vibrancy to tabstrip in light theme.

https://reviewboard.mozilla.org/r/193384/#review198590

What about the dark theme?

Comment 4

[(inactive) Nihanth Subramanya [:nhnt11]]

Attachment: Dark theme screenshot

(In reply to Dão Gottwald [::dao] from comment #3)
> Comment on attachment 8922335 [details]
> Bug 1386721 - Add vibrancy to tabstrip in light theme.
> 
> https://reviewboard.mozilla.org/r/193384/#review198590
> 
> What about the dark theme?

I was just about to comment about this. :P

I don't think we should do this for the dark theme without further discussion. Personally, I think it looks ugly and also kinda goes against the philosophy of the dark theme. See screenshots.

Comment 5

[Dão Gottwald [:dao]]

Looks fine to me. Please consult shorlander.

Comment 6

[(inactive) Nihanth Subramanya [:nhnt11]]

12:30:21 AM GMT+5:30 - shorlander: Yeah I think the problem with the dark theme is that foreground can end up [darker] than the [background].
12:30:41 AM GMT+5:30 - shorlander: Can we try it on just the light theme?

So, let's land this patch as is and update the bug title?

Comment 7

[Dão Gottwald [:dao]]

(In reply to Nihanth Subramanya [:nhnt11] from comment #6)
> 12:30:21 AM GMT+5:30 - shorlander: Yeah I think the problem with the dark
> theme is that foreground can end up [darker] than the [background].

I'm not sure what this means. What's foreground and background here? Navbar + selected tab vs. background tabs? Background tabs are always darker in the dark theme on Windows and Mac...

Comment 8

[Sören Hentzschel]

> I don't think we should do this for the dark theme without further discussion. Personally, I think it looks ugly

I would like to understand this statement. The dark theme looks ugly with vibrance so it shouldn't be implemented, you say. But the default theme has also a dark tab strip and vibrancy was implemented for the default theme. I understand that not everyone like the vibrancy effect (I am also not the biggest fan) but I don't understand why it should be there in the default theme and not in the dark theme because both has a similar tab strip on macOS. I would also understand that it's maybe desirable to have a theme without vibrancy but then I don't understand why the light theme should have vibrancy and the dark theme should not. Maybe an option for enabling/disabling the vibrancy would be great, but this would be another request and I don't think the light theme and the dark theme should be treated differently. ;)

Comment 9

[(inactive) Nihanth Subramanya [:nhnt11]]

(In reply to Sören Hentzschel from comment #8)
> > I don't think we should do this for the dark theme without further discussion. Personally, I think it looks ugly
> 
> I would like to understand this statement. The dark theme looks ugly with
> vibrance so it shouldn't be implemented, you say. But the default theme has
> also a dark tab strip and vibrancy was implemented for the default theme. I
> understand that not everyone like the vibrancy effect (I am also not the
> biggest fan) but I don't understand why it should be there in the default
> theme and not in the dark theme because both has a similar tab strip on
> macOS. I would also understand that it's maybe desirable to have a theme
> without vibrancy but then I don't understand why the light theme should have
> vibrancy and the dark theme should not. Maybe an option for
> enabling/disabling the vibrancy would be great, but this would be another
> request and I don't think the light theme and the dark theme should be
> treated differently. ;)

I don't think the background color of the nav toolbar and selected tab in the dark theme goes well with vibrancy. This is my personal subjective opinion and hence my desire for further discussion. :)

For me, one way to put it would be that vibrancy can mean a tinted background (see attachment 8922338 [details]) which clashes with the dark background of the tabs/navbar (but I don't think it clashes with the bright background used in the default and light themes).

(In reply to Dão Gottwald [::dao] from comment #7)
> (In reply to Nihanth Subramanya [:nhnt11] from comment #6)
> > 12:30:21 AM GMT+5:30 - shorlander: Yeah I think the problem with the dark
> > theme is that foreground can end up [darker] than the [background].
> 
> I'm not sure what this means. What's foreground and background here? Navbar
> + selected tab vs. background tabs? Background tabs are always darker in the
> dark theme on Windows and Mac...

Stephen, could you clarify please?

Comment 10

[Stephen Horlander [:shorlander]]

Sorry, what I meant to say was:

That the Background Tabs & Tabstrip with vibrancy can end-up brighter than the Foreground Tabs & Toolbar. Which visually messes with the z-order we want to establish.

The light theme doesn't seem to have the problem and I think it looks nice there. So we should leave the tabstrip fully opaque for the dark theme.

Comment 11

[Dão Gottwald [:dao]]

(In reply to Stephen Horlander [:shorlander] from comment #10)
> Sorry, what I meant to say was:
> 
> That the Background Tabs & Tabstrip with vibrancy can end-up brighter than
> the Foreground Tabs & Toolbar. Which visually messes with the z-order we
> want to establish.

This is already the case atm with the dark theme without vibrancy...

Comment 12

[(inactive) Nihanth Subramanya [:nhnt11]]

(In reply to Nihanth Subramanya [:nhnt11] from comment #4)
> Created attachment 8922338 [details]
> Dark theme screenshot
> 
> (In reply to Dão Gottwald [::dao] from comment #3)
> > Comment on attachment 8922335 [details]
> > Bug 1386721 - Add vibrancy to tabstrip in light theme.
> > 
> > https://reviewboard.mozilla.org/r/193384/#review198590
> > 
> > What about the dark theme?
> 
> I was just about to comment about this. :P
> 
> I don't think we should do this for the dark theme without further
> discussion. Personally, I think it looks ugly and also kinda goes against
> the philosophy of the dark theme. See screenshots.

Sorry, I wasn't clear about what this screenshot is showing: both cases shown are with vibrancy enabled. In the first one, there's something light behind Firefox, and in the second one, my bright wallpaper is behind Firefox.

Comment 13

[Dão Gottwald [:dao]]

Comment on attachment 8922335 [details]
Bug 1386721 - Add vibrancy to tabstrip in light theme.

https://reviewboard.mozilla.org/r/193384/#review199034

Comment 14

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/b4f2482fac1f
Add vibrancy to tabstrip in light theme. r=dao

Comment 15

[Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)]

https://hg.mozilla.org/mozilla-central/rev/b4f2482fac1f

Comment 16

[ovidiu boca[:Ovidiu]]

I verified this on Mac OS X 10.13 with FF Nightly 58.0a1(2017-11-07) and I can confirm that the vibrancy is added to tabstrip in Light theme.