Web Inspector: Canvas: missing icons for WebGL2 contexts
authordrousso@apple.com <drousso@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sun, 24 Mar 2019 20:21:13 +0000 (20:21 +0000)
committerdrousso@apple.com <drousso@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sun, 24 Mar 2019 20:21:13 +0000 (20:21 +0000)
https://bugs.webkit.org/show_bug.cgi?id=196136

Reviewed by Timothy Hatcher.

* UserInterface/Views/CanvasSidebarPanel.css:
(.sidebar > .panel.navigation.canvas > .navigation-bar > .item.record-start-stop.disabled > .glyph): Added.
(.sidebar > .panel.navigation.canvas > .content > .tree-outline .item.canvas:matches(.canvas-2d, .bitmaprenderer) .icon): Added.
(.sidebar > .panel.navigation.canvas > .content > .tree-outline .item.canvas:matches(.webgl, .webgl2, .webgpu, .webmetal) .icon): Added.
(@media (prefers-color-scheme: dark)):
(.sidebar > .panel.navigation.canvas > .navigation-bar > .item.record-start-stop.disabled): Deleted.
(.sidebar > .panel.navigation.canvas > .content > .tree-outline .item.canvas.canvas-2d .icon): Deleted.
(.sidebar > .panel.navigation.canvas > .content > .tree-outline .item.casnvas.webgl .icon): Deleted.
Use the 2D icon for BitmapRenderer and the 3D icon for WebGL2, WebMetals, and WebGPU.
Drive-by: the start/stop button text was too dark when disabled in dark mode.
* UserInterface/Models/Canvas.js:
Drive-by: "gpu" => "webgpu", to match the protocol string.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@243424 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebInspectorUI/ChangeLog
Source/WebInspectorUI/UserInterface/Models/Canvas.js
Source/WebInspectorUI/UserInterface/Views/CanvasSidebarPanel.css

index 67b1eb2..522cca3 100644 (file)
@@ -1,3 +1,24 @@
+2019-03-24  Devin Rousso  <drousso@apple.com>
+
+        Web Inspector: Canvas: missing icons for WebGL2 contexts
+        https://bugs.webkit.org/show_bug.cgi?id=196136
+
+        Reviewed by Timothy Hatcher.
+
+        * UserInterface/Views/CanvasSidebarPanel.css:
+        (.sidebar > .panel.navigation.canvas > .navigation-bar > .item.record-start-stop.disabled > .glyph): Added.
+        (.sidebar > .panel.navigation.canvas > .content > .tree-outline .item.canvas:matches(.canvas-2d, .bitmaprenderer) .icon): Added.
+        (.sidebar > .panel.navigation.canvas > .content > .tree-outline .item.canvas:matches(.webgl, .webgl2, .webgpu, .webmetal) .icon): Added.
+        (@media (prefers-color-scheme: dark)):
+        (.sidebar > .panel.navigation.canvas > .navigation-bar > .item.record-start-stop.disabled): Deleted.
+        (.sidebar > .panel.navigation.canvas > .content > .tree-outline .item.canvas.canvas-2d .icon): Deleted.
+        (.sidebar > .panel.navigation.canvas > .content > .tree-outline .item.casnvas.webgl .icon): Deleted.
+        Use the 2D icon for BitmapRenderer and the 3D icon for WebGL2, WebMetals, and WebGPU.
+        Drive-by: the start/stop button text was too dark when disabled in dark mode.
+
+        * UserInterface/Models/Canvas.js:
+        Drive-by: "gpu" => "webgpu", to match the protocol string.
+
 2019-03-22  Devin Rousso  <drousso@apple.com>
 
         Web Inspector: Sources: "Reveal in Resources Tab" still shown when Sources tab is enabled
 2019-03-22  Devin Rousso  <drousso@apple.com>
 
         Web Inspector: Sources: "Reveal in Resources Tab" still shown when Sources tab is enabled
index dfaeb9f..7a3880f 100644 (file)
@@ -412,7 +412,7 @@ WI.Canvas.ContextType = {
     BitmapRenderer: "bitmaprenderer",
     WebGL: "webgl",
     WebGL2: "webgl2",
     BitmapRenderer: "bitmaprenderer",
     WebGL: "webgl",
     WebGL2: "webgl2",
-    WebGPU: "gpu",
+    WebGPU: "webgpu",
     WebMetal: "webmetal",
 };
 
     WebMetal: "webmetal",
 };
 
index e30da03..18fcebc 100644 (file)
     overflow-y: hidden;
 }
 
     overflow-y: hidden;
 }
 
-.sidebar > .panel.navigation.canvas > .navigation-bar > .item.record-start-stop.disabled {
+.sidebar > .panel.navigation.canvas > .navigation-bar > .item.record-start-stop.disabled > .glyph {
     filter: grayscale();
     opacity: 0.5;
 }
 
     filter: grayscale();
     opacity: 0.5;
 }
 
-.sidebar > .panel.navigation.canvas > .content > .tree-outline .item.canvas.canvas-2d .icon {
+.sidebar > .panel.navigation.canvas > .content > .tree-outline .item.canvas:matches(.canvas-2d, .bitmaprenderer) .icon {
     content: url(../Images/Canvas2D.svg);
 }
 
     content: url(../Images/Canvas2D.svg);
 }
 
-.sidebar > .panel.navigation.canvas > .content > .tree-outline .item.canvas.webgl .icon {
+.sidebar > .panel.navigation.canvas > .content > .tree-outline .item.canvas:matches(.webgl, .webgl2, .webgpu, .webmetal) .icon {
     content: url(../Images/Canvas3D.svg);
 }
 
     content: url(../Images/Canvas3D.svg);
 }
 
 }
 
 @media (prefers-color-scheme: dark) {
 }
 
 @media (prefers-color-scheme: dark) {
-    .sidebar > .panel.navigation.canvas > .navigation-bar > .item.record-start-stop.disabled {
+    .sidebar > .panel.navigation.canvas > .navigation-bar > .item.record-start-stop.disabled > .glyph {
         filter: grayscale() invert();
     }
 }
         filter: grayscale() invert();
     }
 }