1075672 - [10.10] Style the sidebar in Places window with Yosemite behind-window vibrancy

Status: RESOLVED FIXED

(Firefox :: Theme, defect)

Description

[Stefan [:stefanh]]

Attachment: Style the Places sidebar

Had a chat with Markus about this and thought I file/fix it before it gets lost :-)
Basically the same patch as the one in bug 1059278.

Note: I don't have Yosemite so it's untested.

Comment 1

[:Gijs (he/him)]

Comment on attachment 8498278 [details] [diff] [review]
Style the Places sidebar

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

Tested this, works a charm. Thanks for the patch!

Comment 2

[:Gijs (he/him)]

remote:   https://hg.mozilla.org/integration/fx-team/rev/69bb4035eba1

Comment 3

[Ed Morley [:emorley]]

https://hg.mozilla.org/mozilla-central/rev/69bb4035eba1

Comment 4

[:Gijs (he/him)]

Marco, I think this should be on the spreadsheet.

Comment 5

[:Gijs (he/him)]

Comment on attachment 8498278 [details] [diff] [review]
Style the Places sidebar

Approval Request Comment
[Feature/regressing bug #]: Yosemite
[User impact if declined]: no beautiful sidebars on yosemite in the places window
[Describe test coverage new/current, TBPL]: n/a, style-only
[Risks and why]: very low
[String/UUID change made/needed]: nope

Comment 6

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

Comment on attachment 8498278 [details] [diff] [review]
Style the Places sidebar

Aurora+

Comment 7

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-aurora/rev/690369cf0542