Attachment 357859 Details for Bug 192916 – Patch

[patch] Patch

patch.txt (text/plain), 3.05 KB, created by Nikita Vasilyev on 2018-12-20 13:38:05 PST

(hide)

Description:

Filename:

MIME Type:

Creator: Nikita Vasilyev

Created: 2018-12-20 13:38:05 PST

Size: 3.05 KB

patch

obsolete

>diff --git a/Source/WebInspectorUI/ChangeLog b/Source/WebInspectorUI/ChangeLog
>index b464cca872a..fb48b1cc4b9 100644
>--- a/Source/WebInspectorUI/ChangeLog
>+++ b/Source/WebInspectorUI/ChangeLog
>@@ -1,3 +1,19 @@
>+2018-12-20  Nikita Vasilyev  <nvasilyev@apple.com>
>+
>+        Web Inspector: Dark Mode: Type profiler popovers have black text on dark background
>+        https://bugs.webkit.org/show_bug.cgi?id=192916
>+        <rdar://problem/46863518>
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        * UserInterface/Views/SourceCodeTextEditor.css:
>+        (.popover .debugger-popover-content > .body):
>+        * UserInterface/Views/TypeTreeElement.css:
>+        (.item.type-tree-element.prototype):
>+        (.item.type-tree-element.prototype:hover,):
>+        * UserInterface/Views/TypeTreeView.css:
>+        (.type-tree):
>+
> 2018-12-19  Devin Rousso  <drousso@apple.com>
> 
>         Web Inspector: Audit: fix descriptions for default tests
>diff --git a/Source/WebInspectorUI/UserInterface/Views/SourceCodeTextEditor.css b/Source/WebInspectorUI/UserInterface/Views/SourceCodeTextEditor.css
>index 072afc85bd4..af1c2e08bb1 100644
>--- a/Source/WebInspectorUI/UserInterface/Views/SourceCodeTextEditor.css
>+++ b/Source/WebInspectorUI/UserInterface/Views/SourceCodeTextEditor.css
>@@ -144,7 +144,7 @@
> }
> 
> .popover .debugger-popover-content > .body {
>-    border-top: 1px solid hsla(0, 0%, 0%, 0.3);
>+    border-top: 1px solid var(--text-color-quaternary);
>     overflow: auto;
>     padding-top: 2px;
> }
>diff --git a/Source/WebInspectorUI/UserInterface/Views/TypeTreeElement.css b/Source/WebInspectorUI/UserInterface/Views/TypeTreeElement.css
>index d1f48d1ba81..a14c0cf5658 100644
>--- a/Source/WebInspectorUI/UserInterface/Views/TypeTreeElement.css
>+++ b/Source/WebInspectorUI/UserInterface/Views/TypeTreeElement.css
>@@ -74,8 +74,10 @@ body[dir=rtl] .item.type-tree-element.parent > .disclosure-button {
> .item.type-tree-element.prototype {
>     display: inline-block;
> 
>-    border: 1px solid hsla(0, 0%, 0%, 0.06);
>-    background-color: hsla(0, 0%, 0%, 0.03);
>+    /* Unusual order of values because of <webkit.org/b/192922>. */
>+    border: hsla(0, 0%, var(--foreground-lightness), 0.06) solid 1px;
>+
>+    background-color: hsla(0, 0%, var(--foreground-lightness), 0.03);
>     border-radius: 3px;
> 
>     padding-bottom: 1px;
>@@ -98,7 +100,7 @@ body[dir=rtl] .item.type-tree-element.prototype {
> 
> .item.type-tree-element.prototype:hover,
> .item.type-tree-element.prototype:focus {
>-    border-color: hsla(0, 0%, 0%, 0.1);
>+    border-color: hsla(0, 0%, var(--foreground-lightness), 0.1);
> }
> 
> .item.type-tree-element.prototype + ol {
>diff --git a/Source/WebInspectorUI/UserInterface/Views/TypeTreeView.css b/Source/WebInspectorUI/UserInterface/Views/TypeTreeView.css
>index bfa2ab0792b..e979d7b3a56 100644
>--- a/Source/WebInspectorUI/UserInterface/Views/TypeTreeView.css
>+++ b/Source/WebInspectorUI/UserInterface/Views/TypeTreeView.css
>@@ -27,7 +27,7 @@
>     position: relative;
>     display: block;
> 
>-    color: black;
>+    color: var(--text-color);
>     font-family: -webkit-system-font, sans-serif;
>     font-size: 12px;
> }

Flags:

hi: review+

Actions: View | Formatted Diff | Diff

Attachments on bug 192916: 357783 | 357784 | 357785 | 357799 | 357859 | 357904