Slintpad: resize the cansvas after showing the UI

Right now, the winit window adaptor will force a px value on the width and height once the ui is up. We should then override that again with 100% after the UI is shown
task-jp · Jan 3, 2024 · 2ff9399 · 2ff9399
1 parent a3e3ccc
commit 2ff9399
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 6 deletions.
diff --git a/editors/vscode/src/wasm_preview.ts b/editors/vscode/src/wasm_preview.ts
@@ -164,7 +164,11 @@ function getPreviewHtml(
         }
     });
 
-    preview_connector.show_ui().then(() => vscode.postMessage({ command: 'preview_ready' }));
+    preview_connector.show_ui().then(() => {
+        canvas.style.width = "100%";
+        canvas.style.height = "100%";
+        vscode.postMessage({ command: 'preview_ready' });
+    });
 
     </script>
 </head>

diff --git a/tools/slintpad/src/preview_widget.ts b/tools/slintpad/src/preview_widget.ts
@@ -25,11 +25,6 @@ export class PreviewWidget extends Widget {
         canvas.style.height = "100%";
         canvas.style.outline = "none";
         canvas.style.touchAction = "none";
-        canvas.width = canvas.offsetWidth;
-        canvas.height = canvas.offsetHeight;
-
-        canvas.dataset.slintAutoResizeToPreferred = "false";
-
         node.appendChild(canvas);
 
         return node;
@@ -56,6 +51,9 @@ export class PreviewWidget extends Widget {
             // when searching the document.
             this.#previewer.show_ui().then(() => {
                 console.info("UI should be up!");
+                const canvas = document.getElementById(canvas_id) as HTMLElement;
+                canvas.style.width = "100%";
+                canvas.style.height = "100%";
             });
         });
     }