ocp-viewer-core 1.0.2 → 1.0.4

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ocp-viewer-core",
-  "version": "1.0.2",
+  "version": "1.0.4",
   "description": "Shared viewer policy for ocp_vscode, ocp_viewer, Jupyter CadQuery and build123d Studio",
   "type": "module",
   "main": "src/index.js",

package/src/page.js

@@ -455,7 +455,7 @@ export function createPage({
             }
             // One dispatch for every key, shared with every other host, so
             // that a setter cannot exist in one client and not another.
-            applyConfig(viewer, data.config, {
+            const applied = applyConfig(viewer, data.config, {
                 // Only the host knows the other two dimensions, so a
                 // viewport key is a resize rather than a setter.
                 resize: (key, value) => {
@@ -475,6 +475,26 @@ export function createPage({
                     debugLog(`ui: no setter for '${key}'`);
                 }
             });
+            // The viewer changed; _config follows, for exactly the applied
+            // keys. Without this the two disagree, and resize() - a window
+            // resize, a splitter drag, Studio's pane observer on every Run -
+            // re-derives geometry from the value the ui call just replaced:
+            // set_viewer_config(glass=False) held until the next resize
+            // snapped glass back on. Same family: tools, treeWidth, theme.
+            for (const key of applied) {
+                _config[key] = data.config[key];
+            }
+            // Glass and tools decide whether the tree sits over the canvas or
+            // beside it, and three-cad-viewer re-lays out with the `cadWidth`
+            // it was given: leaving glass mode grows the viewer by `treeWidth`
+            // where the canvas should have shrunk by it, so it overflows the
+            // pane until something else happens to resize it. The host owns
+            // the other two dimensions, so it re-derives all three here, the
+            // way a window resize does - `_config` carries the new value by
+            // now, which is what `normalizeWidth` reads.
+            if (applied.includes("glass") || applied.includes("tools")) {
+                resize();
+            }
         } else if (data.type === "animation") {
             // Explode goes off first: both transform the same objects,
             // and animating an already-displaced model is wrong.

package/src/version.js

@@ -24,4 +24,4 @@
    limitations under the License.
 */
 
-export const VERSION = "1.0.2";
+export const VERSION = "1.0.4";