1421389 - Update dark theme with lower-contrast sidebars

Status: RESOLVED FIXED

(DevTools :: General, defect, P3)

Description

[Victoria Wang [:victoria]]

The following changes should be made to the dark theme. This is a more refined look that makes all the sidebars subtler-looking, which was especially important for the Debugger.

#909090 sidebar/accordion text
#58575c line numbers
#1B1B1D sidebar backgrounds, source header, source outline unselected tab, line number gutter, etc.
#222225 accordion header

Should look like the second example in this image: https://pbs.twimg.com/media/DPLwSvDVQAA03Bi.jpg:large
CC: :gl

Comment 1

[Victoria Wang [:victoria]]

It occurred to me that the sidebars in Inspector (and most panels) are much more important than they are in Debugger, and this will make it a bit hard to read crucial pieces like CSS rule names.

Could we keep the main body text in all non-Debugger panels the same as they currently are (#B1B1B3) and just tone it down to #909090 for the Debugger? New background colors for all panels is fine.

Comment 2

[Victoria Wang [:victoria]]

Just poking this issue :). Any chance of movement on this?

Comment 3

[Gabriel Luong [:gl] (ΦωΦ)]

Attachment: Bug 1421389 - Update dark theme with lower-contrast sidebars.

Review commit: https://reviewboard.mozilla.org/r/214228/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/214228/

Comment 4

[Patrick Brosset <:pbro>]

Comment on attachment 8943882 [details]
Bug 1421389 - Update dark theme with lower-contrast sidebars.

https://reviewboard.mozilla.org/r/214228/#review219956

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/a1e6c367f91c
Update dark theme with lower-contrast sidebars. r=pbro

Comment 6

[Raul Gurzau (:RaulG)]

https://hg.mozilla.org/mozilla-central/rev/a1e6c367f91c