Attachment 372508 Details for Bug 199047 – Patch

[patch] Patch

bug-199047-20190619181119.patch (text/plain), 2.30 KB, created by Matt Baker on 2019-06-19 18:11:20 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: Matt Baker

Created: 2019-06-19 18:11:20 PDT

Size: 2.30 KB

patch

obsolete

>Subversion Revision: 246619
>diff --git a/Source/WebInspectorUI/ChangeLog b/Source/WebInspectorUI/ChangeLog
>index ba07a2c81596cada6b8b7013140f55cb67999820..1675169a27e3d939caf50758514dbc0df686d985 100644
>--- a/Source/WebInspectorUI/ChangeLog
>+++ b/Source/WebInspectorUI/ChangeLog
>@@ -1,3 +1,21 @@
>+2019-06-19  Matt Baker  <mattbaker@apple.com>
>+
>+        Web Inspector: current call frame indicator not visible in dark mode
>+        https://bugs.webkit.org/show_bug.cgi?id=199047
>+        <rdar://problem/51922895>
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        Of the four possible state combinations a selected call frame tree element
>+        can take, (light/dark mode and focused/unfocused), only unfocused dark mode
>+        requires special treatment. The rest can use --selected-foreground-color
>+        (white). This is consistent with Xcode.
>+
>+        * UserInterface/Views/CallFrameTreeElement.css:
>+        (.tree-outline .item.call-frame .status > .status-image):
>+        (.tree-outline .item.call-frame.selected .status > .status-image):
>+        (@media (prefers-color-scheme: dark)):
>+
> 2019-06-19  Matt Baker  <mattbaker@apple.com>
> 
>         Web Inspector: Remove unused _pendingFilter from NetworkTableContentView
>diff --git a/Source/WebInspectorUI/UserInterface/Views/CallFrameTreeElement.css b/Source/WebInspectorUI/UserInterface/Views/CallFrameTreeElement.css
>index 8871001ec9c1b56df56e3b86eb6aea0302179e1c..dea990c655012a57ee3df981c1406b97f4b07dac 100644
>--- a/Source/WebInspectorUI/UserInterface/Views/CallFrameTreeElement.css
>+++ b/Source/WebInspectorUI/UserInterface/Views/CallFrameTreeElement.css
>@@ -51,7 +51,11 @@ body[dir=rtl] .tree-outline .item.call-frame .status {
> .tree-outline .item.call-frame .status > .status-image {
>     width: 17px;
>     height: 17px;
>-    fill: var(--border-color);
>+    display: none;
>+}
>+
>+.tree-outline .item.call-frame.selected .status > .status-image {
>+    display: inline-block;
> }
> 
> .tree-outline .item.call-frame.selected .status > .status-image {
>@@ -101,3 +105,9 @@ body[dir=rtl] .tree-outline .item.call-frame .status {
> .tree-outline.single-thread .item.call-frame.async-boundary::before {
>     width: 20px;
> }
>+
>+@media (prefers-color-scheme: dark) {
>+    .tree-outline .item.call-frame.selected .status > .status-image {
>+        fill: var(--selected-foreground-color);
>+    }
>+}

Actions: View | Formatted Diff | Diff

Attachments on bug 199047: 372507 | 372508 | 372567 | 373532 | 373535