@marimo-team/islands 0.19.7-dev20 → 0.19.7-dev23

package/dist/main.js

@@ -72913,7 +72913,7 @@ Image URL: ${r.imageUrl}`)), contextToXml({
       return Logger.warn("Failed to get version from mount config"), null;
     }
   }
-  const marimoVersionAtom = atom(getVersionFromMountConfig() || "0.19.7-dev20"), showCodeInRunModeAtom = atom(true);
+  const marimoVersionAtom = atom(getVersionFromMountConfig() || "0.19.7-dev23"), showCodeInRunModeAtom = atom(true);
   atom(null);
   var import_compiler_runtime$89 = require_compiler_runtime();
   function useKeydownOnElement(e, r) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@marimo-team/islands",
-  "version": "0.19.7-dev20",
+  "version": "0.19.7-dev23",
   "main": "dist/main.js",
   "types": "dist/index.d.ts",
   "type": "module",

package/src/components/editor/chrome/panels/logs-panel.tsx

@@ -35,14 +35,12 @@ const LogsPanel: React.FC = () => {
   }
 
   return (
-    <>
+    <div className="flex flex-col h-full overflow-auto">
       <div className="flex flex-row justify-start px-2 py-1">
         <ClearButton dataTestId="clear-logs-button" onClick={clearLogs} />
       </div>
-      <div className="overflow-auto flex-1">
-        <LogViewer logs={logs} className="min-w-[300px]" />
-      </div>
-    </>
+      <LogViewer logs={logs} className="min-w-[300px]" />
+    </div>
   );
 };
 

package/src/utils/__tests__/iframe.test.ts

@@ -0,0 +1,139 @@
+/* Copyright 2026 Marimo. All rights reserved. */
+import { afterEach, beforeEach, describe, expect, it, vi } from "vitest";
+import { captureIframeAsImage } from "../iframe";
+
+// Mock html-to-image
+vi.mock("html-to-image", () => ({
+  toPng: vi.fn().mockResolvedValue("data:image/png;base64,mock"),
+}));
+
+describe("captureIframeAsImage", () => {
+  const originalCreateElement = document.createElement.bind(document);
+
+  beforeEach(() => {
+    // Mock devicePixelRatio
+    Object.defineProperty(window, "devicePixelRatio", {
+      value: 1,
+      writable: true,
+    });
+
+    // Mock canvas for placeholder generation
+    vi.spyOn(document, "createElement").mockImplementation((tagName) => {
+      const element = originalCreateElement(tagName);
+      if (tagName === "canvas") {
+        const mockCtx = {
+          scale: vi.fn(),
+          fillStyle: "",
+          fillRect: vi.fn(),
+          strokeStyle: "",
+          strokeRect: vi.fn(),
+          font: "",
+          textAlign: "",
+          textBaseline: "",
+          fillText: vi.fn(),
+          measureText: vi.fn().mockReturnValue({ width: 10 }),
+        };
+        vi.spyOn(element as HTMLCanvasElement, "getContext").mockReturnValue(
+          mockCtx as unknown as CanvasRenderingContext2D,
+        );
+        vi.spyOn(element as HTMLCanvasElement, "toDataURL").mockReturnValue(
+          "data:image/png;base64,placeholder",
+        );
+      }
+      return element;
+    });
+  });
+
+  afterEach(() => {
+    vi.clearAllMocks();
+    vi.restoreAllMocks();
+  });
+
+  it("should return null when element has no iframe", async () => {
+    const element = document.createElement("div");
+    element.innerHTML = "<p>No iframe here</p>";
+
+    const result = await captureIframeAsImage(element);
+    expect(result).toBeNull();
+  });
+
+  it("should return placeholder for external iframe", async () => {
+    const element = document.createElement("div");
+    element.innerHTML = '<iframe src="https://external.com/page"></iframe>';
+
+    const result = await captureIframeAsImage(element);
+
+    expect(result).not.toBeNull();
+    expect(result).toMatch(/^data:image\/png;base64,/);
+  });
+
+  it("should return placeholder for cross-origin iframe", async () => {
+    const element = document.createElement("div");
+    element.innerHTML =
+      '<iframe src="https://www.openstreetmap.org/export/embed.html"></iframe>';
+
+    const result = await captureIframeAsImage(element);
+
+    expect(result).not.toBeNull();
+    expect(result).toMatch(/^data:image\/png;base64,/);
+  });
+
+  it("should return null for about:blank iframe without body", async () => {
+    const element = document.createElement("div");
+    const iframe = document.createElement("iframe");
+    iframe.src = "about:blank";
+    element.append(iframe);
+
+    // The iframe has no body accessible in jsdom
+    const result = await captureIframeAsImage(element);
+
+    // In jsdom, contentDocument may not be accessible
+    expect(result).toBeNull();
+  });
+
+  it("should handle iframe with relative same-origin src", async () => {
+    const element = document.createElement("div");
+    element.innerHTML = '<iframe src="./@file/123.html"></iframe>';
+
+    // Same-origin relative URL, but contentDocument not accessible in jsdom
+    const result = await captureIframeAsImage(element);
+
+    // Returns null because jsdom can't access contentDocument for file URLs
+    expect(result).toBeNull();
+  });
+
+  it("should detect external URL from various formats", async () => {
+    const externalUrls = [
+      "https://example.com",
+      "http://external.org/path",
+      "https://sub.domain.com:8080/page",
+    ];
+
+    for (const url of externalUrls) {
+      const element = document.createElement("div");
+      element.innerHTML = `<iframe src="${url}"></iframe>`;
+
+      const result = await captureIframeAsImage(element);
+
+      expect(result).not.toBeNull();
+      expect(result).toMatch(/^data:image\/png;base64,/);
+    }
+  });
+
+  it("should not treat same-origin URLs as external", async () => {
+    // Same-origin URLs - these should try to capture, not return placeholder
+    const sameOriginUrls = ["/local/path", "./relative/path", "../parent/path"];
+
+    for (const url of sameOriginUrls) {
+      const element = document.createElement("div");
+      element.innerHTML = `<iframe src="${url}"></iframe>`;
+
+      const result = await captureIframeAsImage(element);
+
+      // In jsdom, these return null because contentDocument isn't accessible
+      // but they should NOT return a placeholder (which would indicate external detection)
+      // The key is they don't trigger the external URL path
+      expect(result).toBeNull();
+    }
+  });
+});

package/src/utils/download.ts

@@ -7,6 +7,7 @@ import { Filenames } from "@/utils/filenames";
 import { Paths } from "@/utils/paths";
 import { prettyError } from "./errors";
 import { toPng } from "./html-to-image";
+import { captureIframeAsImage } from "./iframe";
 import { Logger } from "./Logger";
 
 /**
@@ -104,6 +105,12 @@ export async function getImageDataUrlForCell(
   if (!element) {
     return;
   }
+
+  const iframeDataUrl = await captureIframeAsImage(element);
+  if (iframeDataUrl) {
+    return iframeDataUrl;
+  }
+
   const cleanup = prepareCellElementForScreenshot(element, enablePrintMode);
 
   try {
@@ -125,6 +132,13 @@ export async function downloadCellOutputAsImage(
     return;
   }
 
+  // Cell outputs that are iframes
+  const iframeDataUrl = await captureIframeAsImage(element);
+  if (iframeDataUrl) {
+    downloadByURL(iframeDataUrl, Filenames.toPNG(filename));
+    return;
+  }
+
   await downloadHTMLAsImage({
     element,
     filename,

package/src/utils/iframe.ts

@@ -0,0 +1,142 @@
+/* Copyright 2026 Marimo. All rights reserved. */
+
+import { toPng } from "./html-to-image";
+
+const PLACEHOLDER_WIDTH = 320;
+const PLACEHOLDER_HEIGHT = 180;
+
+function isExternalUrl(src: string | null): string | null {
+  if (!src || src === "about:blank") {
+    return null;
+  }
+  try {
+    const resolved = new URL(src, window.location.href);
+    if (resolved.origin === window.location.origin) {
+      return null;
+    }
+    return resolved.href;
+  } catch {
+    return src;
+  }
+}
+
+function wrapText(
+  ctx: CanvasRenderingContext2D,
+  text: string,
+  maxWidth: number,
+): string[] {
+  const lines: string[] = [];
+  let current = "";
+
+  for (const char of text) {
+    const test = current + char;
+    if (ctx.measureText(test).width <= maxWidth) {
+      current = test;
+    } else {
+      if (current) {
+        lines.push(current);
+      }
+      current = char;
+    }
+  }
+  if (current) {
+    lines.push(current);
+  }
+
+  return lines;
+}
+
+function createPlaceholderImage(url: string | null): string {
+  const scale = window.devicePixelRatio || 1;
+  const canvas = document.createElement("canvas");
+  canvas.width = PLACEHOLDER_WIDTH * scale;
+  canvas.height = PLACEHOLDER_HEIGHT * scale;
+
+  const ctx = canvas.getContext("2d");
+  if (!ctx) {
+    return canvas.toDataURL("image/png");
+  }
+
+  ctx.scale(scale, scale);
+
+  // Background
+  ctx.fillStyle = "#f3f4f6";
+  ctx.fillRect(0, 0, PLACEHOLDER_WIDTH, PLACEHOLDER_HEIGHT);
+
+  // Border
+  ctx.strokeStyle = "#d1d5db";
+  ctx.strokeRect(0.5, 0.5, PLACEHOLDER_WIDTH - 1, PLACEHOLDER_HEIGHT - 1);
+
+  // Text
+  ctx.fillStyle = "#6b7280";
+  ctx.font = "8px sans-serif";
+  ctx.textAlign = "center";
+  ctx.textBaseline = "middle";
+
+  const lineHeight = 14;
+  const padding = 16;
+  const maxWidth = PLACEHOLDER_WIDTH - padding * 2;
+
+  const message = "External iframe";
+  const urlLines = url ? wrapText(ctx, url, maxWidth) : [];
+  const totalLines = 1 + urlLines.length;
+  const totalHeight = totalLines * lineHeight;
+  let y = (PLACEHOLDER_HEIGHT - totalHeight) / 2 + lineHeight / 2;
+
+  ctx.fillText(message, PLACEHOLDER_WIDTH / 2, y);
+  y += lineHeight;
+
+  for (const line of urlLines) {
+    ctx.fillText(line, PLACEHOLDER_WIDTH / 2, y);
+    y += lineHeight;
+  }
+
+  return canvas.toDataURL("image/png");
+}
+
+/**
+ * Capture an iframe as a PNG image. We need to do this because external iframes are not supported by html-to-image.
+ * @param element - The element to capture the iframe from
+ * @returns The image data URL of the iframe, or a placeholder image if the iframe is external
+ */
+export async function captureIframeAsImage(
+  element: HTMLElement,
+): Promise<string | null> {
+  const iframe = element.querySelector("iframe");
+  if (!iframe) {
+    return null;
+  }
+
+  // Check if the iframe itself is external
+  const externalUrl = isExternalUrl(iframe.getAttribute("src"));
+  if (externalUrl) {
+    return createPlaceholderImage(externalUrl);
+  }
+
+  // Try to access iframe document and check for nested external iframes
+  let doc: Document;
+  try {
+    const d = iframe.contentDocument || iframe.contentWindow?.document;
+    if (!d?.body) {
+      return null;
+    }
+    doc = d;
+  } catch {
+    return createPlaceholderImage(null);
+  }
+
+  // Check for nested external iframes
+  for (const nested of doc.querySelectorAll("iframe")) {
+    const nestedExternal = isExternalUrl(nested.getAttribute("src"));
+    if (nestedExternal) {
+      return createPlaceholderImage(nestedExternal);
+    }
+  }
+
+  // Capture the iframe content
+  try {
+    return await toPng(doc.body);
+  } catch {
+    return createPlaceholderImage(null);
+  }
+}