metaframe-widget 0.3.3__tar.gz → 0.3.6__tar.gz

{metaframe_widget-0.3.3 → metaframe_widget-0.3.6}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: metaframe-widget
-Version: 0.3.3
+Version: 0.3.6
 Summary: anywidget for embedding metaframe URLs in Jupyter and marimo notebooks
 Project-URL: Homepage, https://github.com/metapages/metaframe-js
 Project-URL: Repository, https://github.com/metapages/metaframe-js

{metaframe_widget-0.3.3 → metaframe_widget-0.3.6}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "metaframe-widget"
-version = "0.3.3"
+version = "0.3.6"
 description = "anywidget for embedding metaframe URLs in Jupyter and marimo notebooks"
 license = "MIT"
 requires-python = ">=3.8"

{metaframe_widget-0.3.3 → metaframe_widget-0.3.6}/src/metaframe_widget/_esm.py

@@ -1,5 +1,5 @@
 ESM = """
-const CDN_URL = "https://cdn.jsdelivr.net/npm/@metapages/metapage@1.10.6/+esm";
+const CDN_URL = "https://cdn.jsdelivr.net/npm/@metapages/metapage@1.10.8/+esm";
 
 let _renderMetapage = null;
 

{metaframe_widget-0.3.3 → metaframe_widget-0.3.6}/src/metaframe_widget/_widget.py

@@ -6,6 +6,15 @@ import traitlets
 
 from ._esm import ESM
 
+import base64
+
+import urllib.parse
+
+# Equivalent to btoa(encodeURIComponent(value));
+def string_to_base64_string(value: str) -> str:
+    encoded_uri_component = urllib.parse.quote(value, safe="-_.!~*'()")
+    return base64.b64encode(encoded_uri_component.encode("ascii")).decode("ascii")
+
 
 class MetaframeWidget(anywidget.AnyWidget):
     """Widget that renders a metaframe iframe (works in Jupyter and marimo).
@@ -28,6 +37,21 @@ class MetaframeWidget(anywidget.AnyWidget):
     height = traitlets.Unicode("400px").tag(sync=True)
     allow = traitlets.Unicode("").tag(sync=True)
 
+    def __init__(self, *args, **kwargs):
+        super().__init__(*args, **kwargs)
+        # Sync layout.height/width so ipywidgets' LayoutView doesn't clear
+        # the styles that the ESM sets on the element.
+        self.layout.height = self.height
+        self.layout.width = self.width
+        self.observe(self._sync_layout_height, names=["height"])
+        self.observe(self._sync_layout_width, names=["width"])
+
+    def _sync_layout_height(self, change):
+        self.layout.height = change["new"]
+
+    def _sync_layout_width(self, change):
+        self.layout.width = change["new"]
+
     def set_inputs(self, d: dict):
         """Merge a dict into the current inputs."""
         self.inputs = {**self.inputs, **d}
@@ -62,8 +86,8 @@ class MetaframeWidget(anywidget.AnyWidget):
             height: CSS height for the widget container (default "400px").
             allow: iframe allow attribute string (e.g. "camera; microphone").
         """
-        encoded = base64.b64encode(js_code.encode()).decode()
-        url = f"https://js.mtfm.io/#?js={urllib.parse.quote(encoded)}"
+        encoded = string_to_base64_string(js_code)
+        url = f"https://js.mtfm.io/#?js={encoded}"
         return cls(url=url, width=width, height=height, allow=allow, **kwargs)
 
     def pipe_to(self, target: "MetaframeWidget", output_key: str, input_key: str = None):

{metaframe_widget-0.3.3 → metaframe_widget-0.3.6}/tests/test_widget.py

@@ -31,8 +31,10 @@ def test_set_input():
 def test_from_code():
     code = 'console.log("hello");'
     w = MetaframeWidget.from_code(code)
-    encoded = base64.b64encode(code.encode()).decode()
-    expected_url = f"https://js.mtfm.io/#?js={urllib.parse.quote(encoded)}"
+    # from_code does: encodeURIComponent(code) then base64
+    encoded_uri = urllib.parse.quote(code, safe="-_.!~*'()")
+    encoded = base64.b64encode(encoded_uri.encode("ascii")).decode("ascii")
+    expected_url = f"https://js.mtfm.io/#?js={encoded}"
     assert w.url == expected_url
 
 
@@ -75,6 +77,33 @@ def test_pipe_to_ignores_unrelated_keys():
     assert sink.inputs == {}
 
 
+def test_layout_height_synced_on_init():
+    """layout.height must match the height trait so ipywidgets doesn't clear it."""
+    w = MetaframeWidget()
+    assert w.layout.height == "400px"
+    assert w.layout.width == "100%"
+
+
+def test_layout_height_synced_with_custom_value():
+    w = MetaframeWidget(height="600px", width="80%")
+    assert w.layout.height == "600px"
+    assert w.layout.width == "80%"
+
+
+def test_layout_height_synced_from_code():
+    w = MetaframeWidget.from_code("console.log(1)", height="420px")
+    assert w.layout.height == "420px"
+
+
+def test_layout_height_updates_on_change():
+    """Changing height trait after init must propagate to layout."""
+    w = MetaframeWidget()
+    w.height = "500px"
+    assert w.layout.height == "500px"
+    w.width = "50%"
+    assert w.layout.width == "50%"
+
+
 def test_pipe_chain_five_widgets():
     """Five widgets piped in a chain propagate data end-to-end."""
     w1 = MetaframeWidget()