Closed
Bug 1000928
Opened 11 years ago
Closed 11 years ago
Dev Tools Themes: Debugger uses low-res gear image
Categories
(DevTools :: Debugger, defect)
DevTools
Debugger
Tracking
(Not tracked)
RESOLVED
FIXED
Firefox 31
People
(Reporter: jryans, Assigned: bgrins)
References
Details
Attachments
(2 files)
237.88 KB,
image/png
|
Details | |
9.76 KB,
patch
|
jryans
:
review+
|
Details | Diff | Splinter Review |
In retina mode on a Mac, the debugger shows a low-res gear icon on the right, but the toolbox options icon is high-res. We should use the high-res one everywhere.
Screenshot attached.
Assignee | ||
Comment 1•11 years ago
|
||
I have some changes that reorganize this in Bug 942292, but it shouldn't necessarily block this.
It still doesn't switch to the SVG icon, but this should be easy to do (use tool-options.svg instead of option-icon.png and delete the option-icon.png file).
Assignee | ||
Updated•11 years ago
|
Assignee | ||
Updated•11 years ago
|
Assignee: nobody → bgrinstead
Assignee | ||
Comment 2•11 years ago
|
||
Can you check this out and see what you think? Notice that there is no blue state available anymore on the vector icon, so I'm changing opacity when it opens instead. It shouldn't be a big problem though - it is pretty obvious when the menu is opened.
Attachment #8411888 -
Flags: review?(jryans)
Assignee | ||
Updated•11 years ago
|
Status: NEW → ASSIGNED
Assignee | ||
Updated•11 years ago
|
Reporter | ||
Comment 3•11 years ago
|
||
Comment on attachment 8411888 [details] [diff] [review]
options-button-vector.patch
Review of attachment 8411888 [details] [diff] [review]:
-----------------------------------------------------------------
Seems great to me! Thanks.
Attachment #8411888 -
Flags: review?(jryans) → review+
Assignee | ||
Comment 4•11 years ago
|
||
https://hg.mozilla.org/integration/fx-team/rev/dbb1d4de276c
https://tbpl.mozilla.org/?tree=Fx-Team&rev=dbb1d4de276c
Whiteboard: [fixed-in-fx-team]
Comment 5•11 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → Firefox 31
Updated•10 years ago
|
QA Whiteboard: [good first verify]
Updated•6 years ago
|
Product: Firefox → DevTools
You need to log in
before you can comment on or make changes to this bug.
Description
•