Closed
Bug 1121432
Opened 9 years ago
Closed 4 years ago
Findbar previous and next button look bad in devedition dark theme
Categories
(DevTools :: General, defect)
DevTools
General
Tracking
(Not tracked)
RESOLVED
WORKSFORME
People
(Reporter: ntim, Unassigned)
References
(Blocks 1 open bug)
Details
(Keywords: regression, Whiteboard: [devedition-polish])
Attachments
(3 files)
No description provided.
Updated•9 years ago
|
Component: Theme → Developer Tools
Summary: Findbar previous and next button look bad in devedition → Findbar previous and next button look bad in devedition dark theme
Reporter | ||
Comment 1•9 years ago
|
||
Assets for anyone who'd like to work on this.
Reporter | ||
Comment 4•9 years ago
|
||
Should be easier to work on once bug 1096413 is resolved.
Depends on: 1096413
Reporter | ||
Updated•9 years ago
|
Assignee: nobody → ntim.bugs
Status: NEW → ASSIGNED
Reporter | ||
Comment 5•9 years ago
|
||
This also fixes bug 1131765.
Attachment #8594641 -
Flags: review?(gijskruitbosch+bugs)
Comment 6•9 years ago
|
||
Comment on attachment 8594641 [details] [diff] [review] Patch Review of attachment 8594641 [details] [diff] [review]: ----------------------------------------------------------------- This uses the inverted close icon also for the light theme, which is wrong. Also, the new arrow images are off-center on Win8, at least. ::: browser/themes/shared/devedition.inc.css @@ +223,5 @@ > + border-radius: 0; > +} > + > +.browserContainer .findbar-textbox[status="notfound"] { > + background-color: #EB5368 !important; /* --theme-highlight-red */ What's with the comment here? :-)
Attachment #8594641 -
Flags: review?(gijskruitbosch+bugs) → review-
Comment 7•9 years ago
|
||
Reporter | ||
Comment 8•9 years ago
|
||
(In reply to :Gijs Kruitbosch from comment #7) > Created attachment 8594697 [details] > off-center-arrows.PNG Not sure why that happens on Windows 8, since I haven't changed the padding at all.
Comment 9•9 years ago
|
||
(In reply to Tim Nguyen [:ntim] from comment #8) > (In reply to :Gijs Kruitbosch from comment #7) > > Created attachment 8594697 [details] > > off-center-arrows.PNG > > Not sure why that happens on Windows 8, since I haven't changed the padding > at all. Are you sure the image size hasn't changed at all?
Reporter | ||
Updated•9 years ago
|
Assignee: ntim.bugs → nobody
Status: ASSIGNED → NEW
Updated•6 years ago
|
Product: Firefox → DevTools
Reporter | ||
Comment 10•4 years ago
|
||
Probably irrelevant at this point, even for the light/dark themes.
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → WORKSFORME
You need to log in
before you can comment on or make changes to this bug.
Description
•