status bar does not show background image from the LW theme

RESOLVED FIXED in Firefox 55

Status

()

RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: aceman, Assigned: Paenglab)

Tracking

Trunk
mozilla55
x86_64
Linux
Points:
---

Firefox Tracking Flags

(firefox55 fixed)

Details

Attachments

(1 attachment)

(Reporter)

Description

2 years ago
Since the recent changes in lightweight themes handling on trunk, the status bar does not show the image from the theme. The toolbars work fine. The statusbar only takes some color from the theme and is drawn with a solid fill of this color.
(Assignee)

Comment 1

2 years ago
Created attachment 8848729 [details] [diff] [review]
bug1348137.patch

Dão, I added a -moz-appearance: none; when a LW-theme is active. But the main work in this patch is the removal of the fallback border and background-color for the statusbar.
Assignee: nobody → richard.marti
Status: NEW → ASSIGNED
Attachment #8848729 - Flags: review?(dao+bmo)

Updated

2 years ago
Attachment #8848729 - Flags: review?(dao+bmo) → review+
(Assignee)

Comment 2

2 years ago
Thank you.
Keywords: checkin-needed
(Reporter)

Comment 3

2 years ago
Thanks, works for me. I had to reapply the theme for it to take effect.

Comment 4

2 years ago
Pushed by archaeopteryx@coole-files.de:
https://hg.mozilla.org/integration/mozilla-inbound/rev/784aca50c657
Remove border and background fallback styling from statusbar. r=dao
Keywords: checkin-needed

Comment 5

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/784aca50c657
Status: ASSIGNED → RESOLVED
Last Resolved: 2 years ago
status-firefox55: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
You need to log in before you can comment on or make changes to this bug.