Closed Bug 1152358 Opened 9 years ago Closed 9 years ago

Linux back/forward button are not styled properly in devedition theme

Categories

(Firefox :: Theme, defect)

Product:
Firefox
Component:
Theme
Platform:
x86
Linux
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
Firefox 40
Tracking Flags:
Tracking Status
firefox40 --- fixed

People

(Reporter: bgrins, Assigned: bgrins)

References

Details

Attachments

(3 files)

unstyled.png
(deleted), image/png
Details
with-patch.png
(deleted), image/png
Details
linux-back-forward.patch
(deleted), patch
Gijs
: review+
Details | Diff | Splinter Review 
There is a selector that still includes :not(:-moz-lwtheme) in linux/devedition.css: https://dxr.mozilla.org/mozilla-central/source/browser/themes/linux/devedition.css#30. 

This is causing the back and forward buttons on linux to not be styled properly
Attached image unstyled.png (deleted) — 
How it looks now
Attached image with-patch.png (deleted) — 
How it looks with the patch applied
Attached patch linux-back-forward.patch (deleted) — — Splinter Review 
Simple fix since DE is now a lw theme
Assignee: nobody → bgrinstead
Status: NEW → ASSIGNED
Attachment #8589681 - Flags: review?(gijskruitbosch+bugs)
Comment on attachment 8589681 [details] [diff] [review]
linux-back-forward.patch

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

rs=me
Attachment #8589681 - Flags: review?(gijskruitbosch+bugs) → review+
https://hg.mozilla.org/mozilla-central/rev/4292c699c0d2
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
status-firefox40: --- → fixed
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → Firefox 40
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: