@marimo-team/islands 0.18.5-dev168 → 0.18.5-dev169

package/src/components/editor/chrome/wrapper/sidebar.tsx

@@ -8,72 +8,105 @@ import { useMemo } from "react";
 import { ReorderableList } from "@/components/ui/reorderable-list";
 import { Tooltip } from "@/components/ui/tooltip";
 import { notebookQueuedOrRunningCountAtom } from "@/core/cells/cells";
-import { snippetsEnabledAtom } from "@/core/config/config";
 import { cn } from "@/utils/cn";
 import { FeedbackButton } from "../components/feedback-button";
-import { sidebarOrderAtom, useChromeActions, useChromeState } from "../state";
+import { panelLayoutAtom, useChromeActions, useChromeState } from "../state";
 import { PANEL_MAP, PANELS, type PanelDescriptor } from "../types";
 
 export const Sidebar: React.FC = () => {
-  const { selectedPanel } = useChromeState();
-  const { toggleApplication } = useChromeActions();
-  const [sidebarOrder, setSidebarOrder] = useAtom(sidebarOrderAtom);
-  const snippetsEnabled = useAtomValue(snippetsEnabledAtom);
+  const { selectedPanel, selectedDeveloperPanelTab } = useChromeState();
+  const { toggleApplication, setSelectedDeveloperPanelTab } =
+    useChromeActions();
+  const [panelLayout, setPanelLayout] = useAtom(panelLayoutAtom);
 
   const renderIcon = ({ Icon }: PanelDescriptor, className?: string) => {
     return <Icon className={cn("h-5 w-5", className)} />;
   };
 
-  // Get all available sidebar panels
-  // Panels with defaultHidden are only available if explicitly enabled (e.g., snippets)
-  const availableSidebarPanels = useMemo(
-    () =>
-      PANELS.filter((p) => {
-        if (p.hidden || p.position !== "sidebar") {
-          return false;
-        }
-        // Show defaultHidden panels only if enabled via config
-        if (p.defaultHidden && p.id === "snippets" && !snippetsEnabled) {
-          return false;
-        }
-        return true;
-      }),
-    [snippetsEnabled],
-  );
+  // Get panels available for sidebar context menu
+  // Only show panels that are NOT in the developer panel
+  const availableSidebarPanels = useMemo(() => {
+    const devPanelIds = new Set(panelLayout.developerPanel);
+    return PANELS.filter((p) => {
+      if (p.hidden) {
+        return false;
+      }
+      // Exclude panels that are in the developer panel
+      if (devPanelIds.has(p.type)) {
+        return false;
+      }
+      return true;
+    });
+  }, [panelLayout.developerPanel]);
+
+  // Convert current sidebar items to PanelDescriptors
+  const sidebarItems = useMemo(() => {
+    return panelLayout.sidebar.flatMap((id) => {
+      const panel = PANEL_MAP.get(id);
+      return panel ? [panel] : [];
+    });
+  }, [panelLayout.sidebar]);
+
+  const handleSetSidebarItems = (items: PanelDescriptor[]) => {
+    setPanelLayout((prev) => ({
+      ...prev,
+      sidebar: items.map((item) => item.type),
+    }));
+  };
+
+  const handleReceive = (item: PanelDescriptor, fromListId: string) => {
+    // Remove from the source list
+    if (fromListId === "developer-panel") {
+      setPanelLayout((prev) => ({
+        ...prev,
+        developerPanel: prev.developerPanel.filter((id) => id !== item.type),
+      }));
 
-  const currentItems = sidebarOrder
-    .map((id) => PANEL_MAP.get(id))
-    .filter(Boolean);
+      // If the moved item was selected in dev panel, select the first remaining item
+      if (selectedDeveloperPanelTab === item.type) {
+        const remainingDevPanels = panelLayout.developerPanel.filter(
+          (id) => id !== item.type,
+        );
+        if (remainingDevPanels.length > 0) {
+          setSelectedDeveloperPanelTab(remainingDevPanels[0]);
+        }
+      }
+    }
 
-  const handleSetValue = (panels: PanelDescriptor[]) => {
-    setSidebarOrder(panels.map((p) => p.id));
+    // Select the dropped item in sidebar
+    toggleApplication(item.type);
   };
 
   return (
     <div className="h-full pt-4 pb-1 px-1 flex flex-col items-start text-muted-foreground text-md select-none no-print text-sm z-50 dark:bg-background print:hidden hide-on-fullscreen">
       <ReorderableList<PanelDescriptor>
-        value={currentItems}
-        setValue={handleSetValue}
+        value={sidebarItems}
+        setValue={handleSetSidebarItems}
+        getKey={(p) => p.type}
         availableItems={availableSidebarPanels}
+        crossListDrag={{
+          dragType: "panels",
+          listId: "sidebar",
+          onReceive: handleReceive,
+        }}
         getItemLabel={(panel) => (
-          <span className="flex items-center gap-2 [">
+          <span className="flex items-center gap-2">
             {renderIcon(panel, "h-4 w-4 text-muted-foreground")}
-            {panel.tooltip}
+            {panel.label}
           </span>
         )}
-        ariaLabel="Reorderable sidebar panels"
+        ariaLabel="Sidebar panels"
         className="flex flex-col gap-0"
-        onAction={(panel) => toggleApplication(panel.id)}
-        renderItem={(panel) => {
-          return (
-            <SidebarItem
-              tooltip={panel.tooltip}
-              selected={selectedPanel === panel.id}
-            >
-              {renderIcon(panel)}
-            </SidebarItem>
-          );
-        }}
+        minItems={0}
+        onAction={(panel) => toggleApplication(panel.type)}
+        renderItem={(panel) => (
+          <SidebarItem
+            tooltip={panel.tooltip}
+            selected={selectedPanel === panel.type}
+          >
+            {renderIcon(panel)}
+          </SidebarItem>
+        )}
       />
       <FeedbackButton>
         <SidebarItem tooltip="Send feedback!" selected={false}>

package/src/components/ui/reorderable-list.tsx

@@ -2,7 +2,12 @@
 
 import type React from "react";
 import { useMemo } from "react";
-import { ListBox, ListBoxItem, useDragAndDrop } from "react-aria-components";
+import {
+  type DropItem,
+  ListBox,
+  ListBoxItem,
+  useDragAndDrop,
+} from "react-aria-components";
 import { Logger } from "@/utils/Logger";
 import {
   ContextMenu,
@@ -12,15 +17,37 @@ import {
 } from "./context-menu";
 import "./reorderable-list.css";
 
-export interface ReorderableListProps<T extends { id: string | number }> {
+interface DragData<T> {
+  itemId: string;
+  sourceListId: string;
+  item: T;
+}
+
+function getDragMimeType(dragType: string): string {
+  return `application/x-reorderable-${dragType}`;
+}
+
+function parseDragData<T>(text: string): DragData<T> | null {
+  try {
+    return JSON.parse(text) as DragData<T>;
+  } catch {
+    return null;
+  }
+}
+
+export interface ReorderableListProps<T> {
   /**
-   * The current list of items
+   * The current list of items.
    */
   value: T[];
   /**
    * Callback when items are reordered
    */
   setValue: (items: T[]) => void;
+  /**
+   * Function to get a unique key for each item. Used for drag-drop and rendering.
+   */
+  getKey: (item: T) => string;
   /**
    * Render function for each item.
    * Note: Avoid interactive elements (buttons) inside - they break drag behavior.
@@ -50,49 +77,148 @@ export interface ReorderableListProps<T extends { id: string | number }> {
    * Additional class name for the list container
    */
   className?: string;
+  /**
+   * Configuration for cross-list drag-drop. When set, items can be dragged
+   * between lists that share the same `dragType`.
+   */
+  crossListDrag?: {
+    /** Identifier that links lists together - same dragType = can share items */
+    dragType: string;
+    /** Unique identifier for this list */
+    listId: string;
+    /**
+     * Callback when an item is received from another list.
+     * At this point, setValue has been called with the new item included,
+     * but the parent component may not have re-rendered yet.
+     * Use this to remove the item from the source list and handle any side effects.
+     */
+    onReceive: (item: T, fromListId: string, insertIndex: number) => void;
+  };
 }
 
 /**
- * A generic reorderable list component using react-aria-components and react-stately.
- * Items can be reordered via drag and drop.
+ * A generic reorderable list component using react-aria-components.
+ * Items can be reordered via drag and drop within the list.
+ *
+ * For cross-list drag-drop, set the same `dragType` on multiple lists
+ * and provide an `onReceive` callback to handle items dropped from other lists.
  *
  * @example
  * ```tsx
- * interface MyItem {
- *   id: string;
- *   name: string;
- * }
- *
- * const [items, setItems] = useState<MyItem[]>([...]);
- *
+ * // Single list reordering
  * <ReorderableList
  *   value={items}
  *   setValue={setItems}
+ *   getKey={(item) => item.id}
  *   renderItem={(item) => <div>{item.name}</div>}
- *   ariaLabel="My reorderable list"
+ * />
+ *
+ * // Cross-list drag-drop
+ * <ReorderableList
+ *   value={sidebarItems}
+ *   setValue={setSidebarItems}
+ *   getKey={(item) => item.type}
+ *   renderItem={(item) => <div>{item.name}</div>}
+ *   crossListDrag={{
+ *     dragType: "panels",
+ *     listId: "sidebar",
+ *     onReceive: (item, fromListId) => {
+ *       // Remove from source list
+ *       setOtherItems(prev => prev.filter(i => i.type !== item.type));
+ *     },
+ *   }}
  * />
  * ```
  */
-export const ReorderableList = <T extends { id: string | number }>({
+export const ReorderableList = <T extends object>({
   value,
   setValue,
+  getKey,
   renderItem,
   onAction,
   availableItems,
-  getItemLabel = (item) => String(item.id),
+  getItemLabel,
   minItems = 1,
   ariaLabel = "Reorderable list",
   className,
+  crossListDrag,
 }: ReorderableListProps<T>) => {
+  const mimeType = crossListDrag
+    ? getDragMimeType(crossListDrag.dragType)
+    : null;
+  const onReceive = crossListDrag?.onReceive;
+
+  // Shared handler for cross-list drops
+  const handleCrossListDrop = async (
+    items: DropItem[],
+    insertIndex: number,
+  ) => {
+    if (!mimeType || !crossListDrag?.listId || !onReceive) {
+      return;
+    }
+
+    for (const dragItem of items) {
+      if (dragItem.kind !== "text" || !dragItem.types.has(mimeType)) {
+        continue;
+      }
+
+      const text = await dragItem.getText(mimeType);
+      const data = parseDragData<T>(text);
+      if (!data) {
+        continue;
+      }
+
+      // Only accept drops from different lists
+      if (data.sourceListId === crossListDrag.listId) {
+        continue;
+      }
+
+      // Skip if item already exists in this list
+      if (value.some((item) => getKey(item) === getKey(data.item))) {
+        continue;
+      }
+
+      // Add to this list and notify parent
+      setValue([
+        ...value.slice(0, insertIndex),
+        data.item,
+        ...value.slice(insertIndex),
+      ]);
+      onReceive(data.item, data.sourceListId, insertIndex);
+    }
+  };
+
   const { dragAndDropHooks } = useDragAndDrop<T>({
-    getItems: (keys) => [...keys].map((key) => ({ "text/plain": String(key) })),
+    getItems: (keys) =>
+      [...keys].map((key) => {
+        const item = value.find((i) => getKey(i) === key);
+        const baseData: Record<string, string> = {
+          "text/plain": String(key),
+        };
+
+        // Add cross-list drag data if dragType is set
+        if (mimeType && crossListDrag?.listId && item) {
+          const dragData: DragData<T> = {
+            itemId: String(key),
+            sourceListId: crossListDrag.listId,
+            item,
+          };
+          baseData[mimeType] = JSON.stringify(dragData);
+        }
+
+        return baseData;
+      }),
+
+    // Accept drops from lists with the same dragType
+    acceptedDragTypes: mimeType ? [mimeType, "text/plain"] : ["text/plain"],
+
     onReorder(e) {
       const keySet = new Set(e.keys);
-      const draggedItems = value.filter((item) => keySet.has(item.id));
-      const remaining = value.filter((item) => !keySet.has(item.id));
+      const draggedItems = value.filter((item) => keySet.has(getKey(item)));
+      const remaining = value.filter((item) => !keySet.has(getKey(item)));
 
       const targetIndex = remaining.findIndex(
-        (item) => item.id === e.target.key,
+        (item) => getKey(item) === e.target.key,
       );
       const insertIndex =
         e.target.dropPosition === "before" ? targetIndex : targetIndex + 1;
@@ -103,19 +229,34 @@ export const ReorderableList = <T extends { id: string | number }>({
         ...remaining.slice(insertIndex),
       ]);
     },
+
+    // Handle drops from other lists (on a specific item)
+    async onInsert(e) {
+      const targetIndex = value.findIndex(
+        (item) => getKey(item) === e.target.key,
+      );
+      const insertIndex =
+        e.target.dropPosition === "before" ? targetIndex : targetIndex + 1;
+      await handleCrossListDrop(e.items, insertIndex);
+    },
+
+    // Handle drops on empty list or root
+    async onRootDrop(e) {
+      await handleCrossListDrop(e.items, value.length);
+    },
   });
 
   // Track which items are currently in the list
-  const currentItemIds = useMemo(
-    () => new Set(value.map((item) => item.id)),
-    [value],
+  const currentItemKeys = useMemo(
+    () => new Set(value.map((item) => getKey(item))),
+    [value, getKey],
   );
 
   const handleToggleItem = (item: T, isChecked: boolean) => {
     if (isChecked) {
       setValue([...value, item]);
     } else if (value.length > minItems) {
-      setValue(value.filter((v) => v.id !== item.id));
+      setValue(value.filter((v) => getKey(v) !== getKey(item)));
     }
   };
 
@@ -124,12 +265,12 @@ export const ReorderableList = <T extends { id: string | number }>({
       return;
     }
 
-    const item = value.find((i) => i.id === key);
+    const item = value.find((i) => getKey(i) === key);
 
     if (!item) {
       Logger.warn("handleAction: item not found for key", {
         key,
-        availableIds: value.map((v) => v.id),
+        availableKeys: value.map((v) => getKey(v)),
       });
       return;
     }
@@ -137,19 +278,36 @@ export const ReorderableList = <T extends { id: string | number }>({
     onAction(item);
   };
 
+  // When list is empty, show a drop zone placeholder
+  const isEmpty = value.length === 0;
+
   const listBox = (
     <ListBox
       aria-label={ariaLabel}
       selectionMode="none"
-      items={value}
       dragAndDropHooks={dragAndDropHooks}
       className={className}
       onAction={handleAction}
     >
-      {(item) => (
-        <ListBoxItem className="active:cursor-grabbing data-[dragging]:opacity-60">
+      {value.map((item) => (
+        <ListBoxItem
+          key={getKey(item)}
+          id={getKey(item)}
+          className="active:cursor-grabbing data-[dragging]:opacity-60 outline-none"
+        >
           {renderItem(item)}
         </ListBoxItem>
+      ))}
+      {/*
+       * When the list is empty, render an invisible placeholder item.
+       * This ensures the ListBox maintains minimum dimensions so users can:
+       * 1. Right-click to access the context menu and add items back
+       * 2. Drag items from another list into this empty list
+       */}
+      {isEmpty && (
+        <ListBoxItem id="__empty__" className="min-h-[40px] min-w-[40px]">
+          <span />
+        </ListBoxItem>
       )}
     </ListBox>
   );
@@ -164,19 +322,20 @@ export const ReorderableList = <T extends { id: string | number }>({
       <ContextMenuTrigger asChild={true}>{listBox}</ContextMenuTrigger>
       <ContextMenuContent>
         {availableItems.map((item) => {
-          const isChecked = currentItemIds.has(item.id);
+          const key = getKey(item);
+          const isChecked = currentItemKeys.has(key);
           const isDisabled = isChecked && value.length <= minItems;
 
           return (
             <ContextMenuCheckboxItem
-              key={item.id}
+              key={key}
               checked={isChecked}
               disabled={isDisabled}
               onCheckedChange={(checked) => {
                 handleToggleItem(item, checked);
               }}
             >
-              {getItemLabel(item)}
+              {getItemLabel ? getItemLabel(item) : key}
             </ContextMenuCheckboxItem>
           );
         })}