Closed Bug 1075672 Opened 5 years ago Closed 5 years ago

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

Categories

(Firefox :: Theme, defect)

x86
macOS
defect
Not set
Points:
1

Tracking

()

RESOLVED FIXED
Firefox 35
Iteration:
35.3
Tracking Status
firefox32 --- wontfix
firefox33 --- wontfix
firefox34 + fixed
firefox35 --- fixed

People

(Reporter: stefanh, Assigned: stefanh)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

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.
Attachment #8498278 - Flags: review?(gijskruitbosch+bugs)
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!
Attachment #8498278 - Flags: review?(gijskruitbosch+bugs) → review+
https://hg.mozilla.org/mozilla-central/rev/69bb4035eba1
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 35
Marco, I think this should be on the spreadsheet.
Iteration: --- → 35.3
Points: --- → 1
Flags: qe-verify-
Flags: needinfo?(mmucci)
Flags: firefox-backlog+
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
Attachment #8498278 - Flags: approval-mozilla-aurora?
Flags: needinfo?(mmucci)
Comment on attachment 8498278 [details] [diff] [review]
Style the Places sidebar

Aurora+
Attachment #8498278 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
You need to log in before you can comment on or make changes to this bug.