[webkit-reviews] review granted: [Bug 186606] Web Inspector: open source Dark Mode : [Attachment 342688] Patch

[bugzilla-daemon at webkit.org]

Timothy Hatcher <timothy at apple.com> has granted Nikita Vasilyev
<nvasilyev at apple.com>'s request for review:
Bug 186606: Web Inspector: open source Dark Mode
https://bugs.webkit.org/show_bug.cgi?id=186606

Attachment 342688: Patch

https://bugs.webkit.org/attachment.cgi?id=342688&action=review




--- Comment #3 from Timothy Hatcher <timothy at apple.com> ---
Comment on attachment 342688
  --> https://bugs.webkit.org/attachment.cgi?id=342688
Patch

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

> Source/WebInspectorUI/UserInterface/Main.html:240
> +    <link rel="stylesheet" href="Views/DarkMode.css">

It would be good to merge the rules into the other files for ease of
maintenance in a future patch.

> Source/WebInspectorUI/UserInterface/Main.html:-243
> -    <link rel="stylesheet"
href="WebKitAdditions/WebInspectorUI/WebInspectorUIAdditions.css">

Might as well keep this line, in case we need it in the future.