status bar does not show background image from the LW theme

RESOLVED FIXED in Firefox 55

Status

()

Toolkit
Themes
RESOLVED FIXED
10 months ago
10 months ago

People

(Reporter: aceman, Assigned: Paenglab)

Tracking

Trunk
mozilla55
x86_64
Linux
Points:
---

Firefox Tracking Flags

(firefox55 fixed)

Details

Attachments

(1 attachment)

(Reporter)

Description

10 months 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

10 months 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

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

Comment 2

10 months ago
Thank you.
Keywords: checkin-needed
(Reporter)

Comment 3

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

Comment 4

10 months 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

10 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/784aca50c657
Status: ASSIGNED → RESOLVED
Last Resolved: 10 months 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.