Bug 194729

Summary: Web Inspector: Dark Mode: commas in CSS selectors are too dim
Product: WebKit Reporter: Nikita Vasilyev <nvasilyev>
Component: Web InspectorAssignee: Nikita Vasilyev <nvasilyev>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, inspector-bugzilla-changes, mattbaker, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: All   
OS: All   
Attachments:
Description Flags
[Image] Bug
none
Patch
none
Patch
none
[Image] With patch applied
none
Patch none

Description Nikita Vasilyev 2019-02-15 16:14:28 PST
Created attachment 362175 [details]
[Image] Bug

Commas in CSS selectors and curly braces around style declarations are barely visible in Dark Mode.
Comment 1 Nikita Vasilyev 2019-02-15 16:18:26 PST
Created attachment 362177 [details]
Patch
Comment 2 Radar WebKit Bug Importer 2019-02-15 16:18:45 PST
<rdar://problem/48128592>
Comment 3 Nikita Vasilyev 2019-02-15 16:31:36 PST
Created attachment 362180 [details]
Patch
Comment 4 Nikita Vasilyev 2019-02-15 16:33:07 PST
Created attachment 362181 [details]
[Image] With patch applied
Comment 5 Matt Baker 2019-02-15 17:02:29 PST
Comment on attachment 362180 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=362180&action=review

> Source/WebInspectorUI/UserInterface/Views/SpreadsheetCSSStyleDeclarationSection.css:33
>      border-bottom: 0.5px solid hsla(0, 0%, 0%, 0.2);

This is so similar to --text-color-quaternary in light mode, which is hsl(0, 0%, 85%). Is it possible to use --text-color-quaternary, and eliminate the @media query for dark mode?
Comment 6 Nikita Vasilyev 2019-02-15 17:15:50 PST
Comment on attachment 362180 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=362180&action=review

>> Source/WebInspectorUI/UserInterface/Views/SpreadsheetCSSStyleDeclarationSection.css:33
>>      border-bottom: 0.5px solid hsla(0, 0%, 0%, 0.2);
> 
> This is so similar to --text-color-quaternary in light mode, which is hsl(0, 0%, 85%). Is it possible to use --text-color-quaternary, and eliminate the @media query for dark mode?

Good point!
Comment 7 Nikita Vasilyev 2019-02-15 17:16:25 PST
Created attachment 362191 [details]
Patch
Comment 8 Matt Baker 2019-02-15 17:37:09 PST
Comment on attachment 362191 [details]
Patch

r=me, looks good!
Comment 9 WebKit Commit Bot 2019-02-15 18:03:38 PST
Comment on attachment 362191 [details]
Patch

Clearing flags on attachment: 362191

Committed r241638: <https://trac.webkit.org/changeset/241638>
Comment 10 WebKit Commit Bot 2019-02-15 18:03:39 PST
All reviewed patches have been landed.  Closing bug.