@marimo-team/islands 0.24.1-dev47 → 0.24.1-dev49

package/dist/{common-DttbfuQx.js → common-CuYfB-UB.js}

@@ -31787,7 +31787,7 @@ function ZG() {
 		return i.warn("Failed to get version from mount config"), null;
 	}
 }
-var QG = ZG() || "0.24.1-dev47", $G = Ti(QG), eK = Ti(!0);
+var QG = ZG() || "0.24.1-dev49", $G = Ti(QG), eK = Ti(!0);
 Ti(null);
 //#endregion
 //#region src/core/meta/code-visibility.ts

package/dist/main.js

@@ -6,7 +6,7 @@ import { n as x, r as S, t as C } from "./copy-DgAphl4k.js";
 import { $t as w, At as T, B as E, C as D, Cn as O, Ct as k, Dn as A, Dt as j, En as M, Et as N, F as P, Ft as F, G as I, Gn as L, Gt as ee, H as te, Hn as R, Ht as ne, I as z, In as re, J as ie, Jn as ae, K as oe, Kn as se, Kt as ce, L as le, Ln as ue, Mn as de, Nn as fe, On as pe, Ot as me, P as he, Pn as ge, Pt as _e, Q as ve, Qn as ye, R as be, Rt as xe, Sn as Se, St as Ce, Tt as we, U as Te, Un as Ee, Ut as De, Vt as Oe, W as ke, Wn as Ae, X as je, Y as Me, Yn as Ne, Yt as Pe, Zn as Fe, _ as Ie, _n as Le, _t as Re, a as ze, an as Be, b as Ve, bn as He, bt as Ue, ct as We, en as Ge, g as Ke, gt as qe, hn as Je, i as Ye, in as Xe, it as Ze, j as Qe, jn as $e, jt as et, k as tt, mn as nt, n as rt, nn as it, nt as at, o as ot, ot as st, pn as ct, q as lt, rn as ut, rt as dt, st as ft, tn as pt, un as mt, v as ht, vt as gt, xn as _t, xt as vt, y as yt, yn as bt, yt as xt, z as St, zn as Ct } from "./html-to-image-hKcfSWj_.js";
 import { o as wt, s as Tt, u as Et } from "./dist-DTSuOdDc.js";
 import { D as Dt, E as Ot, O as kt, S as At, T as jt, _ as Mt, a as Nt, c as Pt, d as Ft, f as It, g as Lt, h as Rt, i as zt, l as Bt, m as Vt, n as Ht, o as Ut, p as Wt, r as Gt, s as Kt, t as qt, u as Jt, v as Yt, w as Xt, x as Zt, y as Qt } from "./spec-DqUrBQpS.js";
-import { $ as $t, $t as en, A as tn, At as nn, B as rn, Bt as an, C as on, Ct as sn, D as cn, Dt as ln, E as un, Et as dn, F as fn, Ft as pn, G as mn, Gt as hn, H as gn, Ht as _n, I as vn, It as yn, Jt as bn, K as xn, Kt as Sn, L as Cn, Lt as wn, M as Tn, Mt as En, N as Dn, Nt as On, O as kn, Ot as An, P as jn, Pt as Mn, Q as Nn, Qt as Pn, R as Fn, Rt as In, S as Ln, St as Rn, T as zn, Tt as Bn, U as Vn, Ut as Hn, V as Un, Vt as Wn, W as Gn, Wt as Kn, Xt as qn, Yt as Jn, Z as Yn, Zt as Xn, _ as Zn, an as Qn, at as $n, b as er, bt as tr, c as nr, cn as rr, ct as ir, d as ar, dn as or, dt as sr, en as cr, et as lr, f as ur, fn as dr, ft as fr, g as pr, gt as mr, h as hr, ht as gr, i as _r, it as vr, j as yr, jt as br, k as xr, kt as Sr, ln as Cr, lt as wr, m as Tr, mt as Er, n as Dr, nn as Or, nt as kr, on as Ar, ot as jr, p as Mr, pn as Nr, pt as Pr, q as Fr, qt as Ir, r as Lr, rn as Rr, rt as zr, sn as Br, st as Vr, t as Hr, tn as Ur, tt as Wr, u as Gr, un as Kr, ut as qr, v as Jr, w as Yr, wt as Xr, x as Zr, xt as Qr, y as $r, yt as ei, z as ti, zt as ni } from "./common-DttbfuQx.js";
+import { $ as $t, $t as en, A as tn, At as nn, B as rn, Bt as an, C as on, Ct as sn, D as cn, Dt as ln, E as un, Et as dn, F as fn, Ft as pn, G as mn, Gt as hn, H as gn, Ht as _n, I as vn, It as yn, Jt as bn, K as xn, Kt as Sn, L as Cn, Lt as wn, M as Tn, Mt as En, N as Dn, Nt as On, O as kn, Ot as An, P as jn, Pt as Mn, Q as Nn, Qt as Pn, R as Fn, Rt as In, S as Ln, St as Rn, T as zn, Tt as Bn, U as Vn, Ut as Hn, V as Un, Vt as Wn, W as Gn, Wt as Kn, Xt as qn, Yt as Jn, Z as Yn, Zt as Xn, _ as Zn, an as Qn, at as $n, b as er, bt as tr, c as nr, cn as rr, ct as ir, d as ar, dn as or, dt as sr, en as cr, et as lr, f as ur, fn as dr, ft as fr, g as pr, gt as mr, h as hr, ht as gr, i as _r, it as vr, j as yr, jt as br, k as xr, kt as Sr, ln as Cr, lt as wr, m as Tr, mt as Er, n as Dr, nn as Or, nt as kr, on as Ar, ot as jr, p as Mr, pn as Nr, pt as Pr, q as Fr, qt as Ir, r as Lr, rn as Rr, rt as zr, sn as Br, st as Vr, t as Hr, tn as Ur, tt as Wr, u as Gr, un as Kr, ut as qr, v as Jr, w as Yr, wt as Xr, x as Zr, xt as Qr, y as $r, yt as ei, z as ti, zt as ni } from "./common-CuYfB-UB.js";
 import { c as ri, i as ii, n as ai, o as oi, t as si } from "./useLifecycle-D-zb2Ha5.js";
 import { t as ci } from "./check-DIJJHY-E.js";
 import { A as li, C as ui, D as di, E as fi, F as pi, I as mi, M as hi, N as gi, P as _i, R as vi, S as yi, a as bi, b as xi, c as Si, i as Ci, j as wi, l as Ti, n as Ei, o as Di, r as Oi, s as ki, t as Ai, w as ji, x as Mi } from "./select-Bakm9EUs.js";
@@ -35256,7 +35256,7 @@ function Gq(e) {
 }
 //#endregion
 //#region src/components/editor/renderers/slides-layout/slides-layout.tsx
-var Kq = q.lazy(() => import("./reveal-component-KZ-Xt-Rk.js")), qq = (e) => {
+var Kq = q.lazy(() => import("./reveal-component-C7Dc57LE.js")), qq = (e) => {
 	let t = (0, Y.c)(33), { layout: n, setLayout: r, cells: i, mode: a } = e, o = Nm(k), s = a === "read" || o, c = Nm(lt) > 1, [l, u] = (0, q.useState)(null), d;
 	t[0] !== i || t[1] !== n ? (d = UU(i, n), t[0] = i, t[1] = n, t[2] = d) : d = t[2];
 	let { slideCells: f, skippedIds: p, noOutputIds: m, slideTypes: h, startCellIndex: g } = d, _;

package/dist/{reveal-component-KZ-Xt-Rk.js → reveal-component-C7Dc57LE.js}

@@ -4,7 +4,7 @@ import { t as o } from "./react-DMWcHin-.js";
 import { t as s } from "./compiler-runtime-CAH9RLw2.js";
 import { Ct as c, rt as l } from "./html-to-image-hKcfSWj_.js";
 import { u } from "./dist-DTSuOdDc.js";
-import { J as d, X as f, Y as p, in as m, l as h, n as g, rn as _, sn as v, t as y, u as b } from "./common-DttbfuQx.js";
+import { J as d, X as f, Y as p, in as m, l as h, n as g, rn as _, sn as v, t as y, u as b } from "./common-CuYfB-UB.js";
 import { R as x } from "./select-Bakm9EUs.js";
 import { X as S } from "./dropdown-menu-DEu759Oe.js";
 import { t as C } from "./jsx-runtime-CpoKNRgu.js";

package/package.json

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

package/src/components/editor/actions/types.ts

@@ -45,8 +45,7 @@ export function flattenActions(
   prevLabel = "",
 ): ActionButton[] {
   return actions.flatMap((action) => {
-    // If label is empty, hide
-    if (!action.label) {
+    if (!action.label || action.hidden) {
       return [];
     }
     if (isParentAction(action)) {

package/src/components/editor/actions/useNotebookActions.tsx

@@ -195,6 +195,7 @@ export function useNotebookActions({
     {
       icon: <DownloadIcon size={14} strokeWidth={1.5} />,
       label: "Download",
+      redundant: true,
       handle: NOOP_HANDLER,
       dropdown: [
         {

package/src/components/editor/controls/__tests__/notebook-menu-dropdown.test.tsx

@@ -31,7 +31,9 @@ import {
   exportOptionsAtom,
   lastExportFormatAtom,
 } from "../../actions/export-dialog/state";
+import CommandPalette from "../command-palette";
 import { NotebookMenuDropdown } from "../notebook-menu-dropdown";
+import { commandPaletteAtom } from "../state";
 
 const POSTGRES_SOURCE: DetectedDataSource = {
   id: "postgres-libpq-environment",
@@ -66,6 +68,10 @@ vi.mock("@/core/wasm/utils", async (importOriginal) => {
   return { ...actual, isWasm: vi.fn(() => false) };
 });
 
+global.HTMLElement.prototype.scrollIntoView = () => {
+  // jsdom does not implement scrollIntoView; cmdk calls it on selection.
+};
+
 function wrapper({ children }: { children: React.ReactNode }) {
   return (
     <Provider store={store}>
@@ -83,14 +89,19 @@ function openNotebookMenu() {
   });
 }
 
-async function openDownloadMenu() {
-  openNotebookMenu();
-  fireEvent.click(await screen.findByRole("menuitem", { name: "Download" }));
+function renderNotebookControls() {
+  return render(
+    <>
+      <NotebookMenuDropdown />
+      <CommandPalette />
+    </>,
+    { wrapper },
+  );
 }
 
-async function selectDownload(name: string | RegExp) {
-  await openDownloadMenu();
-  fireEvent.click(await screen.findByRole("menuitem", { name }));
+async function selectDownloadCommand(name: string) {
+  act(() => store.set(commandPaletteAtom, true));
+  fireEvent.click(await screen.findByText(`Download > ${name}`));
 }
 
 describe("NotebookMenuDropdown", () => {
@@ -108,6 +119,7 @@ describe("NotebookMenuDropdown", () => {
     });
     store.set(exportOptionsAtom, DEFAULT_EXPORT_OPTIONS);
     store.set(lastExportFormatAtom, "html");
+    store.set(commandPaletteAtom, false);
     vi.mocked(isWasm).mockReturnValue(false);
     vi.stubGlobal("PointerEvent", MouseEvent);
     vi.stubGlobal("matchMedia", () => ({
@@ -203,10 +215,22 @@ describe("NotebookMenuDropdown", () => {
     await waitFor(() => expect(menuButton).toHaveFocus());
   });
 
+  it("hides the download menu but keeps its command palette actions", async () => {
+    renderNotebookControls();
+
+    openNotebookMenu();
+    expect(
+      screen.queryByRole("menuitem", { name: "Download" }),
+    ).not.toBeInTheDocument();
+
+    act(() => store.set(commandPaletteAtom, true));
+    expect(await screen.findByText("Download > Download as PNG")).toBeVisible();
+  });
+
   it("opens the HTML shortcut with code excluded", async () => {
-    render(<NotebookMenuDropdown />, { wrapper });
+    renderNotebookControls();
 
-    await selectDownload("Download as HTML (exclude code)");
+    await selectDownloadCommand("Download as HTML (exclude code)");
 
     expect(await screen.findByTestId("export-dialog")).toBeVisible();
     expect(screen.getByTestId("export-format-html")).toHaveAttribute(
@@ -223,19 +247,9 @@ describe("NotebookMenuDropdown", () => {
       selectedLayout: "slides",
       layoutData: {},
     });
-    render(<NotebookMenuDropdown />, { wrapper });
+    renderNotebookControls();
 
-    await openDownloadMenu();
-    fireEvent.click(
-      await screen.findByRole("menuitem", {
-        name: "Download as PDF",
-      }),
-    );
-    fireEvent.click(
-      await screen.findByRole("menuitem", {
-        name: /Slides Layout/,
-      }),
-    );
+    await selectDownloadCommand("Download as PDF > Slides Layout");
 
     expect(await screen.findByTestId("export-dialog")).toBeVisible();
     expect(screen.getByTestId("export-format-pdf")).toHaveAttribute(
@@ -251,27 +265,22 @@ describe("NotebookMenuDropdown", () => {
       selectedLayout: "slides",
       layoutData: {},
     });
-    render(<NotebookMenuDropdown />, { wrapper });
+    renderNotebookControls();
 
-    await openDownloadMenu();
-    fireEvent.click(
-      await screen.findByRole("menuitem", {
-        name: "Download as PDF",
-      }),
-    );
+    act(() => store.set(commandPaletteAtom, true));
 
     expect(
-      screen.getByRole("menuitem", { name: "Document Layout" }),
+      await screen.findByText("Download > Download as PDF > Document Layout"),
     ).toBeVisible();
     expect(
-      screen.queryByRole("menuitem", { name: /Slides Layout/ }),
+      screen.queryByText("Download > Download as PDF > Slides Layout"),
     ).not.toBeInTheDocument();
   });
 
   it("preselects each Python format from its download shortcut", async () => {
-    render(<NotebookMenuDropdown />, { wrapper });
+    renderNotebookControls();
 
-    await selectDownload("Download notebook source");
+    await selectDownloadCommand("Download notebook source");
 
     expect(await screen.findByTestId("export-dialog")).toBeVisible();
     expect(screen.getByTestId("export-format-script")).toHaveAttribute(
@@ -287,7 +296,7 @@ describe("NotebookMenuDropdown", () => {
       expect(screen.queryByTestId("export-dialog")).not.toBeInTheDocument(),
     );
 
-    await selectDownload("Download flat script");
+    await selectDownloadCommand("Download flat script");
 
     expect(await screen.findByTestId("export-dialog")).toBeVisible();
     expect(screen.getByRole("radio", { name: "Flat script" })).toBeChecked();