gds-lens 1.0.1 → 1.0.2

package/CHANGELOG.md

@@ -7,6 +7,32 @@ follows [semantic versioning](https://semver.org/spec/v2.0.0.html).
 From 1.0.0 on, a breaking change to the element's API waits for a major
 version. Before that, `0.x` releases changed it freely.
 
+## [1.0.2] - 2026-08-25
+
+### Changed
+
+- Marker-file warnings are spelled out rather than counted. The `⚠ N warnings`
+  row is a folder now, with one wrapping row per warning inside it. The
+  sentences used to live only in a hover title and a `?gdsDebug=1` console
+  line, so the one thing the row existed to report -- a marker may be in the
+  wrong place, a value was not drawn -- was the one thing it did not say.
+
+### Fixed
+
+- Right-click -> "Copy coordinate" did nothing. The menu is dismissed by a
+  `pointerdown` listener on `window`, which asked whether the menu contained
+  `event.target` -- but the viewer moved into a shadow root in 1.0.0, and an
+  event from inside one is retargeted to the `<gds-lens>` host by the time it
+  reaches `window`. The menu therefore hid itself (`display: none`) on the very
+  press that was landing on its own item, so the button never became a click.
+  The menu opened and showed the right coordinate throughout, which is what
+  made it look alive.
+- The keyboard guard that keeps `m`, `h` and `/` from reaching the viewer while
+  someone is typing in one of lil-gui's text boxes, broken by the same
+  retargeting: it saw the host element rather than the focused input for every
+  keystroke, so typing in a filter box could switch modes, toggle the hierarchy
+  tree, or move focus.
+
 ## [1.0.1] - 2026-08-25
 
 ### Fixed
@@ -302,7 +328,8 @@ web page rather than for a webview.
   worker-loading route and shipped unsubstituted. The `createWorker` host hook
   replaces it.
 
-[Unreleased]: https://github.com/EthanLowenthal/GDS-Lens/compare/v1.0.1...HEAD
+[Unreleased]: https://github.com/EthanLowenthal/GDS-Lens/compare/v1.0.2...HEAD
+[1.0.2]: https://github.com/EthanLowenthal/GDS-Lens/compare/v1.0.1...v1.0.2
 [1.0.1]: https://github.com/EthanLowenthal/GDS-Lens/compare/v1.0.0...v1.0.1
 [1.0.0]: https://github.com/EthanLowenthal/GDS-Lens/compare/v0.1.1...v1.0.0
 [0.1.1]: https://github.com/EthanLowenthal/GDS-Lens/compare/v0.1.0...v0.1.1

package/dist/esm/gds-lens.js

@@ -1052,6 +1052,17 @@ var init_viewer = __esm({
         .lil-gui .lil-controller.marker-selected button { background: var(--select-bg) !important; }
         .lil-gui .lil-controller.marker-selected .lil-name { color: var(--select-fg); }
         .lil-gui .lil-controller.marker-more-row { opacity: 0.55; pointer-events: none; }
+        /* Warning rows inside the marker browser's "\u26A0 N warnings" folder.
+           A warning is a sentence, not a value, so the row drops lil-gui's
+           fixed widget height and single line and wraps instead -- a clipped
+           warning is no more use than the bare count was. Inert: there is
+           nothing to click. */
+        .lil-gui .lil-controller.marker-warning-row { pointer-events: none; }
+        .lil-gui .lil-controller.marker-warning-row button { height: auto; min-height: var(--widget-height);
+                                                             padding: 4px 7px; text-align: left;
+                                                             color: var(--danger-dim); background: none; }
+        .lil-gui .lil-controller.marker-warning-row .lil-name { white-space: normal; overflow-wrap: anywhere;
+                                                                line-height: 1.35; }
         /* ---- Hierarchy tree (see renderHierarchy in viewer.js) ----
            The design's cell tree, down the left edge. Like the lil-gui panel
            on the right it floats over the canvas rather than taking width
@@ -4038,9 +4049,17 @@ No top cell places this one, so the tree has no branch that reaches it.`;
     markersFolder.open();
     if (model.warnings.length > 0) {
       fail("[GDS] marker warnings:", model.warnings.join(" | "));
-      const row = markersFolder.add({ w: () => {
-      } }, "w").name(`\u26A0 ${model.warnings.length} warning${model.warnings.length === 1 ? "" : "s"}`);
-      row.domElement.title = model.warnings.join("\n");
+      const warningsFolder = markersFolder.addFolder(
+        `\u26A0 ${model.warnings.length} warning${model.warnings.length === 1 ? "" : "s"}`
+      );
+      warningsFolder.close();
+      warningsFolder.domElement.title = model.warnings.join("\n");
+      for (const warning of model.warnings) {
+        const row = warningsFolder.add({ w: () => {
+        } }, "w").name(warning);
+        row.domElement.classList.add("marker-warning-row");
+        row.domElement.title = warning;
+      }
     }
     const opacityController = markersFolder.add(markerUiState, "opacity", 0, 1, 0.05).name("Opacity").onChange((value) => modulePromise.then((Module) => Module.setMarkerOpacity(value)));
     opacityController.domElement.title = "Opacity of the whole marker overlay";
@@ -4142,7 +4161,7 @@ No top cell places this one, so the tree has no branch that reaches it.`;
       );
     });
     window.addEventListener("pointerdown", (event) => {
-      if (!canvasMenuEl.contains(event.target)) hideCanvasMenu();
+      if (!event.composedPath().includes(canvasMenuEl)) hideCanvasMenu();
     }, true);
     window.addEventListener("wheel", hideCanvasMenu, { passive: true });
     window.addEventListener("resize", hideCanvasMenu);
@@ -4150,7 +4169,7 @@ No top cell places this one, so the tree has no branch that reaches it.`;
   }
   window.addEventListener("keydown", (event) => {
     if (!hasKeyboard()) return;
-    const t = event.target;
+    const t = event.composedPath()[0] || event.target;
     const tag = t && t.tagName;
     if (tag === "TEXTAREA" || tag === "INPUT" && t.type !== "checkbox") return;
     if (event.key === "[") stepMarker(-1);

package/dist/inline-wasm/gds-lens.js

@@ -1028,6 +1028,17 @@ Usually a truncated or half-written file -- a copy that was interrupted, or a do
         .lil-gui .lil-controller.marker-selected button { background: var(--select-bg) !important; }
         .lil-gui .lil-controller.marker-selected .lil-name { color: var(--select-fg); }
         .lil-gui .lil-controller.marker-more-row { opacity: 0.55; pointer-events: none; }
+        /* Warning rows inside the marker browser's "\u26A0 N warnings" folder.
+           A warning is a sentence, not a value, so the row drops lil-gui's
+           fixed widget height and single line and wraps instead -- a clipped
+           warning is no more use than the bare count was. Inert: there is
+           nothing to click. */
+        .lil-gui .lil-controller.marker-warning-row { pointer-events: none; }
+        .lil-gui .lil-controller.marker-warning-row button { height: auto; min-height: var(--widget-height);
+                                                             padding: 4px 7px; text-align: left;
+                                                             color: var(--danger-dim); background: none; }
+        .lil-gui .lil-controller.marker-warning-row .lil-name { white-space: normal; overflow-wrap: anywhere;
+                                                                line-height: 1.35; }
         /* ---- Hierarchy tree (see renderHierarchy in viewer.js) ----
            The design's cell tree, down the left edge. Like the lil-gui panel
            on the right it floats over the canvas rather than taking width
@@ -4014,9 +4025,17 @@ No top cell places this one, so the tree has no branch that reaches it.`;
       markersFolder.open();
       if (model.warnings.length > 0) {
         fail("[GDS] marker warnings:", model.warnings.join(" | "));
-        const row = markersFolder.add({ w: () => {
-        } }, "w").name(`\u26A0 ${model.warnings.length} warning${model.warnings.length === 1 ? "" : "s"}`);
-        row.domElement.title = model.warnings.join("\n");
+        const warningsFolder = markersFolder.addFolder(
+          `\u26A0 ${model.warnings.length} warning${model.warnings.length === 1 ? "" : "s"}`
+        );
+        warningsFolder.close();
+        warningsFolder.domElement.title = model.warnings.join("\n");
+        for (const warning of model.warnings) {
+          const row = warningsFolder.add({ w: () => {
+          } }, "w").name(warning);
+          row.domElement.classList.add("marker-warning-row");
+          row.domElement.title = warning;
+        }
       }
       const opacityController = markersFolder.add(markerUiState, "opacity", 0, 1, 0.05).name("Opacity").onChange((value) => modulePromise.then((Module) => Module.setMarkerOpacity(value)));
       opacityController.domElement.title = "Opacity of the whole marker overlay";
@@ -4118,7 +4137,7 @@ No top cell places this one, so the tree has no branch that reaches it.`;
         );
       });
       window.addEventListener("pointerdown", (event) => {
-        if (!canvasMenuEl.contains(event.target)) hideCanvasMenu();
+        if (!event.composedPath().includes(canvasMenuEl)) hideCanvasMenu();
       }, true);
       window.addEventListener("wheel", hideCanvasMenu, { passive: true });
       window.addEventListener("resize", hideCanvasMenu);
@@ -4126,7 +4145,7 @@ No top cell places this one, so the tree has no branch that reaches it.`;
     }
     window.addEventListener("keydown", (event) => {
       if (!hasKeyboard()) return;
-      const t = event.target;
+      const t = event.composedPath()[0] || event.target;
       const tag = t && t.tagName;
       if (tag === "TEXTAREA" || tag === "INPUT" && t.type !== "checkbox") return;
       if (event.key === "[") stepMarker(-1);

package/dist/web/gds-lens.js

@@ -1028,6 +1028,17 @@ Usually a truncated or half-written file -- a copy that was interrupted, or a do
         .lil-gui .lil-controller.marker-selected button { background: var(--select-bg) !important; }
         .lil-gui .lil-controller.marker-selected .lil-name { color: var(--select-fg); }
         .lil-gui .lil-controller.marker-more-row { opacity: 0.55; pointer-events: none; }
+        /* Warning rows inside the marker browser's "\u26A0 N warnings" folder.
+           A warning is a sentence, not a value, so the row drops lil-gui's
+           fixed widget height and single line and wraps instead -- a clipped
+           warning is no more use than the bare count was. Inert: there is
+           nothing to click. */
+        .lil-gui .lil-controller.marker-warning-row { pointer-events: none; }
+        .lil-gui .lil-controller.marker-warning-row button { height: auto; min-height: var(--widget-height);
+                                                             padding: 4px 7px; text-align: left;
+                                                             color: var(--danger-dim); background: none; }
+        .lil-gui .lil-controller.marker-warning-row .lil-name { white-space: normal; overflow-wrap: anywhere;
+                                                                line-height: 1.35; }
         /* ---- Hierarchy tree (see renderHierarchy in viewer.js) ----
            The design's cell tree, down the left edge. Like the lil-gui panel
            on the right it floats over the canvas rather than taking width
@@ -4014,9 +4025,17 @@ No top cell places this one, so the tree has no branch that reaches it.`;
       markersFolder.open();
       if (model.warnings.length > 0) {
         fail("[GDS] marker warnings:", model.warnings.join(" | "));
-        const row = markersFolder.add({ w: () => {
-        } }, "w").name(`\u26A0 ${model.warnings.length} warning${model.warnings.length === 1 ? "" : "s"}`);
-        row.domElement.title = model.warnings.join("\n");
+        const warningsFolder = markersFolder.addFolder(
+          `\u26A0 ${model.warnings.length} warning${model.warnings.length === 1 ? "" : "s"}`
+        );
+        warningsFolder.close();
+        warningsFolder.domElement.title = model.warnings.join("\n");
+        for (const warning of model.warnings) {
+          const row = warningsFolder.add({ w: () => {
+          } }, "w").name(warning);
+          row.domElement.classList.add("marker-warning-row");
+          row.domElement.title = warning;
+        }
       }
       const opacityController = markersFolder.add(markerUiState, "opacity", 0, 1, 0.05).name("Opacity").onChange((value) => modulePromise.then((Module) => Module.setMarkerOpacity(value)));
       opacityController.domElement.title = "Opacity of the whole marker overlay";
@@ -4118,7 +4137,7 @@ No top cell places this one, so the tree has no branch that reaches it.`;
         );
       });
       window.addEventListener("pointerdown", (event) => {
-        if (!canvasMenuEl.contains(event.target)) hideCanvasMenu();
+        if (!event.composedPath().includes(canvasMenuEl)) hideCanvasMenu();
       }, true);
       window.addEventListener("wheel", hideCanvasMenu, { passive: true });
       window.addEventListener("resize", hideCanvasMenu);
@@ -4126,7 +4145,7 @@ No top cell places this one, so the tree has no branch that reaches it.`;
     }
     window.addEventListener("keydown", (event) => {
       if (!hasKeyboard()) return;
-      const t = event.target;
+      const t = event.composedPath()[0] || event.target;
       const tag = t && t.tagName;
       if (tag === "TEXTAREA" || tag === "INPUT" && t.type !== "checkbox") return;
       if (event.key === "[") stepMarker(-1);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "gds-lens",
-  "version": "1.0.1",
+  "version": "1.0.2",
   "description": "GDSII & OASIS layout parsing and WebGL2 rendering for the browser",
   "license": "MIT",
   "author": "Ethan Lowenthal",

package/src/viewer.css

@@ -267,6 +267,17 @@
         .lil-gui .lil-controller.marker-selected button { background: var(--select-bg) !important; }
         .lil-gui .lil-controller.marker-selected .lil-name { color: var(--select-fg); }
         .lil-gui .lil-controller.marker-more-row { opacity: 0.55; pointer-events: none; }
+        /* Warning rows inside the marker browser's "⚠ N warnings" folder.
+           A warning is a sentence, not a value, so the row drops lil-gui's
+           fixed widget height and single line and wraps instead -- a clipped
+           warning is no more use than the bare count was. Inert: there is
+           nothing to click. */
+        .lil-gui .lil-controller.marker-warning-row { pointer-events: none; }
+        .lil-gui .lil-controller.marker-warning-row button { height: auto; min-height: var(--widget-height);
+                                                             padding: 4px 7px; text-align: left;
+                                                             color: var(--danger-dim); background: none; }
+        .lil-gui .lil-controller.marker-warning-row .lil-name { white-space: normal; overflow-wrap: anywhere;
+                                                                line-height: 1.35; }
         /* ---- Hierarchy tree (see renderHierarchy in viewer.js) ----
            The design's cell tree, down the left edge. Like the lil-gui panel
            on the right it floats over the canvas rather than taking width

package/src/viewer.js

@@ -1776,9 +1776,20 @@ export function createViewer(mountTarget) {
 
         if (model.warnings.length > 0) {
             fail("[GDS] marker warnings:", model.warnings.join(" | "));
-            const row = markersFolder.add({ w: () => {} }, "w")
-                .name(`⚠ ${model.warnings.length} warning${model.warnings.length === 1 ? "" : "s"}`);
-            row.domElement.title = model.warnings.join("\n");
+            // A count on its own is a dead end -- "2 warnings" does not say
+            // whether the file lost half its results or merely used a value
+            // type we draw as text. So the count is a folder and the sentences
+            // are inside it, rather than living only in a hover title and a
+            // console line nobody has open.
+            const warningsFolder = markersFolder.addFolder(
+                `⚠ ${model.warnings.length} warning${model.warnings.length === 1 ? "" : "s"}`);
+            warningsFolder.close();
+            warningsFolder.domElement.title = model.warnings.join("\n");
+            for (const warning of model.warnings) {
+                const row = warningsFolder.add({ w: () => {} }, "w").name(warning);
+                row.domElement.classList.add("marker-warning-row");
+                row.domElement.title = warning;
+            }
         }
 
         const opacityController = markersFolder.add(markerUiState, "opacity", 0, 1, 0.05).name("Opacity")
@@ -1948,7 +1959,13 @@ export function createViewer(mountTarget) {
         // on it), zooming or panning the layout out from under it, and leaving the
         // webview. Escape is handled with the other keys below.
         window.addEventListener("pointerdown", (event) => {
-            if (!canvasMenuEl.contains(event.target)) hideCanvasMenu();
+            // composedPath() rather than event.target: this listener sits on
+            // window, outside the shadow root, so by the time a pointerdown
+            // inside the viewer arrives here it has been retargeted to the
+            // <gds-lens> host -- the menu never "contains" it. That closed the
+            // menu (display: none) on the press that was landing on the menu
+            // item, so the button never got a click and the copy never ran.
+            if (!event.composedPath().includes(canvasMenuEl)) hideCanvasMenu();
         }, true);
         window.addEventListener("wheel", hideCanvasMenu, { passive: true });
         window.addEventListener("resize", hideCanvasMenu);
@@ -1967,7 +1984,11 @@ export function createViewer(mountTarget) {
         if (!hasKeyboard()) return;
         // Don't hijack typing in lil-gui's text/number inputs -- but focused
         // checkboxes and buttons must not block marker stepping.
-        const t = event.target;
+        // composedPath()[0], for the same retargeting reason as the canvas
+        // menu's dismissal above: event.target here is the <gds-lens> host, so
+        // this guard used to see "GDS-LENS" for every keystroke and let m/h//
+        // through while someone was typing in a lil-gui text box.
+        const t = event.composedPath()[0] || event.target;
         const tag = t && t.tagName;
         if (tag === "TEXTAREA" || (tag === "INPUT" && t.type !== "checkbox")) return;
         if (event.key === "[") stepMarker(-1);