1079372 - Make the default browser notification bar yellow

Status: VERIFIED FIXED

(Firefox :: General, defect)

Description

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

The notification bar we use now to ask users to set Firefox as their default browser is currently black. Since this is a pretty important prompt, we should make it more prominent.
The original design specified yellow as the background color, so let's do that.

Comment 1

[:Gijs (he/him)]

Why spend time doing this rather than bug 1025182?

Comment 2

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

(In reply to :Gijs Kruitbosch from comment #1)
> Why spend time doing this rather than bug 1025182?

I was under the impression that bug 1025182 was a general update of the appearance of all notification bars.
This is more of a hotfix to make that specific bar more prominent. There are some concerns about the bar hurting our default browser numbers (that concerns currently have no basis in data), but if they do turn out to be correct, it would be good to have something that we can uplift (hence only changing the color, not the styling).

Comment 3

[Dão Gottwald [:dao]]

Attachment: patch

Comment 4

[:Gijs (he/him)]

Comment on attachment 8501288 [details] [diff] [review]
patch

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

Stealing, rs=me

Comment 5

[Dão Gottwald [:dao]]

https://hg.mozilla.org/integration/fx-team/rev/75046b047a6c

Comment 6

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/75046b047a6c

Comment 7

[Camelia Badau [:cbadau], Desktop Test Engineering]

Attachment: issue.png

Verified on latest Nightly 35.0a1 (buildID: 20141009030201):
- on Windows 7 64bit, Mac OSX 10.9.5 and Mac OSX 10.10 -> the default browser notification bar is yellow
- on Ubuntu 13.10 32bit and Ubuntu 13.04 64bit -> the default browser notification bar is BLACK. Please see attached screenshot "issue.png".

Comment 8

[Dão Gottwald [:dao]]

(In reply to Camelia Badau, QA [:cbadau] from comment #7)
> Created attachment 8503089 [details]
> issue.png
> 
> Verified on latest Nightly 35.0a1 (buildID: 20141009030201):
> - on Windows 7 64bit, Mac OSX 10.9.5 and Mac OSX 10.10 -> the default
> browser notification bar is yellow
> - on Ubuntu 13.10 32bit and Ubuntu 13.04 64bit -> the default browser
> notification bar is BLACK. Please see attached screenshot "issue.png".

This is expected, as the notification bar uses the InfoBackground color on Windows and Linux, which will be yellow with most OS themes but can also resolve to a different color.

On Ubuntu this patch made us go from gray to black; the latter stands out more than the former.

Comment 9

[Camelia Badau [:cbadau], Desktop Test Engineering]

Thank you!

Comment 10

[Dão Gottwald [:dao]]

Comment on attachment 8501288 [details] [diff] [review]
patch

Approval Request Comment
[Feature/regressing bug #]: bug 951627
[User impact if declined]: less prominent default browser notification bar
[Describe test coverage new/current, TBPL]: none
[Risks and why]: one-liner, not risky
[String/UUID change made/needed]: none

Comment 11

[Lawrence Mandel [:lmandel] (use needinfo)]

Comment on attachment 8501288 [details] [diff] [review]
patch

Aurora+

Dão - We have no sheriff coverage this weekend. Can you land this on Aurora before the merge on Monday?

Comment 12

[Dão Gottwald [:dao]]

https://hg.mozilla.org/releases/mozilla-aurora/rev/f6ccaf5a4b15

Comment 13

[[SV Manager] Florin Mezei, QA (:FlorinMezei)]

Verified on latest Aurora 34 (BuildID=20141013004002) with same results as in comment 7: yellow notification on Win 7 x64 and Mac OS X 10.9.5, black notification on Ubuntu 13.04 x64.