Adjust new devtools theme colors and palette on wiki

RESOLVED FIXED in Firefox 42

Status

()

Firefox
Developer Tools: Framework
P3
normal
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: bgrins, Assigned: bgrins)

Tracking

({dev-doc-complete})

unspecified
Firefox 42
x86
Mac OS X
dev-doc-complete
Points:
---

Firefox Tracking Flags

(firefox42 fixed)

Details

(Whiteboard: [polish-backlog])

Attachments

(3 attachments)

(Assignee)

Description

3 years ago
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

3 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

3 years ago
Whiteboard: [devedition-40]
Setting devedition-40 bugs to p3, filter on FB20EAC4-3FC3-48D9-B15F-0587C3987C25
Priority: -- → P3
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)
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

3 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

3 years ago
Created attachment 8628537 [details] [diff] [review]
bluegrey.patch
Attachment #8628537 - Flags: review?(gabriel.luong)
(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

3 years ago
Attachment #8628537 - Flags: review?(gabriel.luong) → review+
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.
https://hg.mozilla.org/mozilla-central/rev/69d9848dd6a5
Status: ASSIGNED → RESOLVED
Last Resolved: 3 years ago
status-firefox42: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 42
(Assignee)

Comment 15

3 years ago
Updated https://developer.mozilla.org/en-US/docs/Tools/DevToolsColors with the new color
Keywords: dev-doc-complete
Whiteboard: [devedition-40] → [polish-backlog]
You need to log in before you can comment on or make changes to this bug.