Closed Bug 1348137 Opened 7 years ago Closed 7 years ago

status bar does not show background image from the LW theme

Categories

(Toolkit :: Themes, enhancement)

x86_64
Linux
enhancement
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla55
Tracking Status
firefox55 --- fixed

People

(Reporter: aceman, Assigned: Paenglab)

Details

Attachments

(1 file)

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.
Attached patch bug1348137.patchSplinter Review
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)
Attachment #8848729 - Flags: review?(dao+bmo) → review+
Thank you.
Keywords: checkin-needed
Thanks, works for me. I had to reapply the theme for it to take effect.
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
https://hg.mozilla.org/mozilla-central/rev/784aca50c657
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: