Closed
Bug 1150578
Opened 10 years ago
Closed 9 years ago
Adjust new devtools theme colors and palette on wiki
Categories
(DevTools :: Framework, defect, P3)
Tracking
(firefox42 fixed)
RESOLVED
FIXED
Firefox 42
Tracking | Status | |
---|---|---|
firefox42 | --- | fixed |
People
(Reporter: bgrins, Assigned: bgrins)
References
Details
(Keywords: dev-doc-complete, Whiteboard: [polish-backlog])
Attachments
(3 files)
Once we land the new colors in 947242, we should make any necessary adjustments to the color palette also update the wiki at the same time https://developer.mozilla.org/en-US/docs/Tools/DevToolsColors.
Assignee | ||
Comment 1•10 years ago
|
||
I have a suggestion in https://bugzilla.mozilla.org/show_bug.cgi?id=947242#c42 to update this color:
Light theme (Blue Grey) from #5f88b0 to #0072AB
Not stuck on the particular color we change to, but I think anything in that direction is an improvement. See the screenshot: https://bug947242.bugzilla.mozilla.org/attachment.cgi?id=8475326.
Assignee | ||
Updated•10 years ago
|
Whiteboard: [devedition-40]
Comment 2•10 years ago
|
||
Setting devedition-40 bugs to p3, filter on FB20EAC4-3FC3-48D9-B15F-0587C3987C25
Priority: -- → P3
Comment 3•10 years ago
|
||
Comment 4•10 years ago
|
||
Comment 5•10 years ago
|
||
With the colors documented on MDN landing in the actual tools' CSS in bug 947242, and the performance graphs now utilizing the colors defined there, we get some weird results. While I prefer the new colors for the text and highlights in most tools, they seem a bit bold/dark when used in our performance renderings and graphs. See the two attachments.
Dark looks pretty good, IMO, but light has the bold/dark look to it. Would it make sense to have a new set of colors, one for text, one for highlights like this? For example, I'd like many of the light theme "highlight" colors to be brighter, but that'd make all the text using these colors pretty hard to read.
Some of these graphs are shipping soon in Fx40, so we'll have to figure out something soon!
Flags: needinfo?(shorlander)
Comment 6•10 years ago
|
||
Adding new colors in bug 1164552 for performance graphs -- can change or add to wiki in the future, but this needs done for 40.1. Clearing ni?
Flags: needinfo?(shorlander)
Assignee | ||
Comment 7•9 years ago
|
||
Stephen, can you take a quick look at https://bug947242.bugzilla.mozilla.org/attachment.cgi?id=8475326 and confirm that the suggested color change is good?
Assignee: nobody → bgrinstead
Status: NEW → ASSIGNED
Flags: needinfo?(shorlander)
Assignee | ||
Comment 8•9 years ago
|
||
Attachment #8628537 -
Flags: review?(gabriel.luong)
Comment 9•9 years ago
|
||
Assignee | ||
Comment 10•9 years ago
|
||
(In reply to Gabriel Luong [:gl] from comment #9)
> https://treeherder.mozilla.org/#/jobs?repo=try&revision=2d4a898d5ce9
Correct try link is here: https://treeherder.mozilla.org/#/jobs?repo=try&revision=bf7877e9a18a
Comment 11•9 years ago
|
||
(In reply to Brian Grinstead [:bgrins] from comment #7)
> Stephen, can you take a quick look at
> https://bug947242.bugzilla.mozilla.org/attachment.cgi?id=8475326 and confirm
> that the suggested color change is good?
Maybe a little close to the other blue, but it's definitely easier to read.
Flags: needinfo?(shorlander)
Updated•9 years ago
|
Attachment #8628537 -
Flags: review?(gabriel.luong) → review+
Comment 13•9 years ago
|
||
We want to file a bug to address the color of the attributes since the new color is a bit close to the blue in the attributes. @shorlander was okay with the changes to land for now.
Comment 14•9 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
status-firefox42:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 42
Assignee | ||
Comment 15•9 years ago
|
||
Updated https://developer.mozilla.org/en-US/docs/Tools/DevToolsColors with the new color
Keywords: dev-doc-complete
Updated•9 years ago
|
Whiteboard: [devedition-40] → [polish-backlog]
Updated•6 years ago
|
Product: Firefox → DevTools
You need to log in
before you can comment on or make changes to this bug.
Description
•