@perses-dev/dashboards 0.10.0 → 0.12.0

package/dist/cjs/components/Dashboard.js

@@ -24,15 +24,14 @@ const _components = require("@perses-dev/components");
 const _context = require("../context");
 const _gridLayout = require("./GridLayout");
 function Dashboard(props) {
-    const { layouts  } = (0, _context.useLayouts)();
+    const panelGroupIds = (0, _context.usePanelGroupIds)();
     return /*#__PURE__*/ (0, _jsxRuntime.jsx)(_material.Box, {
         ...props,
         children: /*#__PURE__*/ (0, _jsxRuntime.jsx)(_components.ErrorBoundary, {
             FallbackComponent: _components.ErrorAlert,
-            children: layouts.map((layout, groupIndex)=>/*#__PURE__*/ (0, _jsxRuntime.jsx)(_gridLayout.GridLayout, {
-                    groupIndex: groupIndex,
-                    groupDefinition: layout
-                }, layout.id))
+            children: panelGroupIds.map((panelGroupId)=>/*#__PURE__*/ (0, _jsxRuntime.jsx)(_gridLayout.GridLayout, {
+                    panelGroupId: panelGroupId
+                }, panelGroupId))
         })
     });
 }

package/dist/cjs/components/DashboardToolbar.js

@@ -19,6 +19,7 @@ Object.defineProperty(exports, "DashboardToolbar", {
     get: ()=>DashboardToolbar
 });
 const _jsxRuntime = require("react/jsx-runtime");
+const _react = require("react");
 const _material = require("@mui/material");
 const _pencilOutline = /*#__PURE__*/ _interopRequireDefault(require("mdi-material-ui/PencilOutline"));
 const _plusBoxOutline = /*#__PURE__*/ _interopRequireDefault(require("mdi-material-ui/PlusBoxOutline"));
@@ -34,12 +35,23 @@ function _interopRequireDefault(obj) {
 const DashboardToolbar = (props)=>{
     const { dashboardName  } = props;
     const { isEditMode , setEditMode  } = (0, _context.useEditMode)();
-    const { openPanelGroupDialog  } = (0, _context.usePanelGroupDialog)();
-    const { addPanel  } = (0, _context.usePanels)();
+    const { openAddPanelGroup , openAddPanel , save  } = (0, _context.useDashboardActions)();
+    const isLaptopSize = (0, _material.useMediaQuery)((0, _material.useTheme)().breakpoints.up('sm'));
+    const [originalSpec, setOriginalSpec] = (0, _react.useState)(undefined);
+    const { spec , resetSpec  } = (0, _context.useDashboardSpec)();
     const onEditButtonClick = ()=>{
+        setOriginalSpec(spec);
         setEditMode(true);
     };
     const onCancelButtonClick = ()=>{
+        // Reset to the original spec and exit edit mode
+        if (originalSpec) {
+            resetSpec(originalSpec);
+        }
+        setEditMode(false);
+    };
+    const onSave = ()=>{
+        save();
         setEditMode(false);
     };
     return /*#__PURE__*/ (0, _jsxRuntime.jsx)(_jsxRuntime.Fragment, {
@@ -70,6 +82,7 @@ const DashboardToolbar = (props)=>{
                                 children: [
                                     /*#__PURE__*/ (0, _jsxRuntime.jsx)(_material.Button, {
                                         variant: "contained",
+                                        onClick: onSave,
                                         children: "Save"
                                     }),
                                     /*#__PURE__*/ (0, _jsxRuntime.jsx)(_material.Button, {
@@ -103,12 +116,12 @@ const DashboardToolbar = (props)=>{
                             children: [
                                 /*#__PURE__*/ (0, _jsxRuntime.jsx)(_material.Button, {
                                     startIcon: /*#__PURE__*/ (0, _jsxRuntime.jsx)(_plusBoxOutline.default, {}),
-                                    onClick: ()=>openPanelGroupDialog(),
+                                    onClick: openAddPanelGroup,
                                     children: "Add Panel Group"
                                 }),
                                 /*#__PURE__*/ (0, _jsxRuntime.jsx)(_material.Button, {
                                     startIcon: /*#__PURE__*/ (0, _jsxRuntime.jsx)(_chartBoxPlusOutline.default, {}),
-                                    onClick: ()=>addPanel(0),
+                                    onClick: openAddPanel,
                                     children: "Add Panel"
                                 }),
                                 /*#__PURE__*/ (0, _jsxRuntime.jsx)(_components1.TimeRangeControls, {})
@@ -139,7 +152,7 @@ const DashboardToolbar = (props)=>{
                             },
                             children: [
                                 /*#__PURE__*/ (0, _jsxRuntime.jsx)(_components1.TimeRangeControls, {}),
-                                /*#__PURE__*/ (0, _jsxRuntime.jsx)(_material.Button, {
+                                isLaptopSize && /*#__PURE__*/ (0, _jsxRuntime.jsx)(_material.Button, {
                                     variant: "outlined",
                                     startIcon: /*#__PURE__*/ (0, _jsxRuntime.jsx)(_pencilOutline.default, {}),
                                     onClick: onEditButtonClick,

package/dist/cjs/components/GridLayout/GridItemContent.js

@@ -19,21 +19,13 @@ Object.defineProperty(exports, "GridItemContent", {
     get: ()=>GridItemContent
 });
 const _jsxRuntime = require("react/jsx-runtime");
-const _core = require("@perses-dev/core");
 const _context = require("../../context");
 const _panel = require("../Panel/Panel");
 function GridItemContent(props) {
-    const { content , groupIndex , itemIndex  } = props;
-    // Find the panel referenced in content in the store
-    const { panels  } = (0, _context.usePanels)();
-    const panelKey = (0, _core.getPanelKeyFromRef)(content);
-    const panelDefinition = panels[panelKey];
-    if (panelDefinition === undefined) {
-        throw new Error(`Panel with key '${panelKey}' was not found`);
-    }
+    const { panelGroupItemId  } = props;
+    const panelDefinition = (0, _context.usePanel)(panelGroupItemId);
     return /*#__PURE__*/ (0, _jsxRuntime.jsx)(_panel.Panel, {
         definition: panelDefinition,
-        groupIndex: groupIndex,
-        itemIndex: itemIndex
+        panelGroupItemId: panelGroupItemId
     });
 }

package/dist/cjs/components/GridLayout/GridLayout.js

@@ -29,7 +29,10 @@ const _gridTitle = require("./GridTitle");
 const _gridItemContent = require("./GridItemContent");
 const ResponsiveGridLayout = (0, _reactGridLayout.WidthProvider)(_reactGridLayout.Responsive);
 function GridLayout(props) {
-    const { groupIndex , groupDefinition , ...others } = props;
+    const { panelGroupId , ...others } = props;
+    const theme = (0, _material.useTheme)();
+    const groupDefinition = (0, _context.usePanelGroup)(panelGroupId);
+    const { updatePanelGroupLayouts  } = (0, _context.usePanelGroupActions)(panelGroupId);
     var ref;
     const [isOpen, setIsOpen] = (0, _react.useState)((ref = !groupDefinition.isCollapsed) !== null && ref !== void 0 ? ref : true);
     const { isEditMode  } = (0, _context.useEditMode)();
@@ -48,7 +51,7 @@ function GridLayout(props) {
                 },
                 children: [
                     groupDefinition.title !== undefined && /*#__PURE__*/ (0, _jsxRuntime.jsx)(_gridTitle.GridTitle, {
-                        groupIndex: groupIndex,
+                        panelGroupId: panelGroupId,
                         title: groupDefinition.title,
                         collapse: groupDefinition.isCollapsed === undefined ? undefined : {
                             isOpen,
@@ -61,17 +64,11 @@ function GridLayout(props) {
                         children: /*#__PURE__*/ (0, _jsxRuntime.jsx)(ResponsiveGridLayout, {
                             className: "layout",
                             breakpoints: {
-                                lg: 1200,
-                                md: 996,
-                                sm: 768,
-                                xs: 480,
+                                sm: theme.breakpoints.values.sm,
                                 xxs: 0
                             },
                             cols: {
-                                lg: 24,
-                                md: 24,
                                 sm: 24,
-                                xs: 24,
                                 xxs: 2
                             },
                             rowHeight: 30,
@@ -81,22 +78,25 @@ function GridLayout(props) {
                             ],
                             isDraggable: isEditMode,
                             isResizable: isEditMode,
-                            children: groupDefinition.items.map(({ x , y , width , height , content  }, itemIndex)=>/*#__PURE__*/ (0, _jsxRuntime.jsx)("div", {
-                                    "data-grid": {
-                                        x,
-                                        y,
-                                        w: width,
-                                        h: height
-                                    },
+                            containerPadding: [
+                                0,
+                                10
+                            ],
+                            layouts: {
+                                sm: groupDefinition.itemLayouts
+                            },
+                            onLayoutChange: updatePanelGroupLayouts,
+                            children: groupDefinition.itemLayouts.map(({ i  })=>/*#__PURE__*/ (0, _jsxRuntime.jsx)("div", {
                                     children: /*#__PURE__*/ (0, _jsxRuntime.jsx)(_components.ErrorBoundary, {
                                         FallbackComponent: _components.ErrorAlert,
                                         children: /*#__PURE__*/ (0, _jsxRuntime.jsx)(_gridItemContent.GridItemContent, {
-                                            groupIndex: groupIndex,
-                                            itemIndex: itemIndex,
-                                            content: content
+                                            panelGroupItemId: {
+                                                panelGroupId,
+                                                panelGroupItemLayoutId: i
+                                            }
                                         })
                                     })
-                                }, itemIndex))
+                                }, i))
                         })
                     })
                 ]

package/dist/cjs/components/GridLayout/GridTitle.js

@@ -35,12 +35,10 @@ function _interopRequireDefault(obj) {
     };
 }
 function GridTitle(props) {
-    const { groupIndex , title , collapse  } = props;
+    const { panelGroupId , title , collapse  } = props;
     const [isHovered, setIsHovered] = (0, _react.useState)(false);
-    const { openPanelGroupDialog , openDeletePanelGroupDialog  } = (0, _context.usePanelGroupDialog)();
-    const { addPanel  } = (0, _context.usePanels)();
+    const { openAddPanel , openEditPanelGroup , deletePanelGroup , moveUp , moveDown  } = (0, _context.usePanelGroupActions)(panelGroupId);
     const { isEditMode  } = (0, _context.useEditMode)();
-    const { layouts , swapPanelGroups  } = (0, _context.useLayouts)();
     const text = /*#__PURE__*/ (0, _jsxRuntime.jsx)(_material.Typography, {
         variant: "h2",
         sx: {
@@ -73,29 +71,29 @@ function GridTitle(props) {
                     children: [
                         /*#__PURE__*/ (0, _jsxRuntime.jsx)(_material.IconButton, {
                             "aria-label": "add panel to group",
-                            onClick: ()=>addPanel(groupIndex),
+                            onClick: openAddPanel,
                             children: /*#__PURE__*/ (0, _jsxRuntime.jsx)(_plus.default, {})
                         }),
                         /*#__PURE__*/ (0, _jsxRuntime.jsx)(_material.IconButton, {
                             "aria-label": "edit group",
-                            onClick: ()=>openPanelGroupDialog(groupIndex),
+                            onClick: openEditPanelGroup,
                             children: /*#__PURE__*/ (0, _jsxRuntime.jsx)(_pencilOutline.default, {})
                         }),
                         /*#__PURE__*/ (0, _jsxRuntime.jsx)(_material.IconButton, {
                             "aria-label": "delete group",
-                            onClick: ()=>openDeletePanelGroupDialog(groupIndex),
+                            onClick: deletePanelGroup,
                             children: /*#__PURE__*/ (0, _jsxRuntime.jsx)(_deleteOutline.default, {})
                         }),
                         /*#__PURE__*/ (0, _jsxRuntime.jsx)(_material.IconButton, {
                             "aria-label": "move group down",
-                            disabled: groupIndex === layouts.length - 1,
-                            onClick: ()=>swapPanelGroups(groupIndex, groupIndex + 1),
+                            disabled: moveDown === undefined,
+                            onClick: moveDown,
                             children: /*#__PURE__*/ (0, _jsxRuntime.jsx)(_arrowDown.default, {})
                         }),
                         /*#__PURE__*/ (0, _jsxRuntime.jsx)(_material.IconButton, {
                             "aria-label": "move group up",
-                            disabled: groupIndex === 0,
-                            onClick: ()=>swapPanelGroups(groupIndex, groupIndex - 1),
+                            disabled: moveUp === undefined,
+                            onClick: moveUp,
                             children: /*#__PURE__*/ (0, _jsxRuntime.jsx)(_arrowUp.default, {})
                         })
                     ]

package/dist/cjs/components/Panel/DeletePanelDialog.js

@@ -28,7 +28,7 @@ function _interopRequireDefault(obj) {
     };
 }
 const DeletePanelDialog = ()=>{
-    const { deletePanelDialog , closeDeletePanelDialog  } = (0, _context.usePanels)();
+    const { deletePanelDialog , closeDeletePanelDialog  } = (0, _context.useDeletePanelDialog)();
     return /*#__PURE__*/ (0, _jsxRuntime.jsxs)(_material.Dialog, {
         open: deletePanelDialog !== undefined,
         children: [
@@ -52,25 +52,27 @@ const DeletePanelDialog = ()=>{
     });
 };
 const DeletePanelForm = ({ deletePanelDialog  })=>{
-    var ref;
-    const { layouts  } = (0, _context.useLayouts)();
-    const { deletePanels , closeDeletePanelDialog  } = (0, _context.usePanels)();
+    const { deletePanel , closeDeletePanelDialog  } = (0, _context.useDeletePanelDialog)();
     const handleDelete = (e)=>{
         e.preventDefault();
-        const { layoutItem  } = deletePanelDialog;
-        deletePanels([
-            layoutItem
-        ]);
+        const { panelGroupItemId  } = deletePanelDialog;
+        deletePanel(panelGroupItemId);
         closeDeletePanelDialog();
     };
     return /*#__PURE__*/ (0, _jsxRuntime.jsxs)("form", {
         onSubmit: handleDelete,
         children: [
-            /*#__PURE__*/ (0, _jsxRuntime.jsx)(_material.DialogContent, {
+            /*#__PURE__*/ (0, _jsxRuntime.jsxs)(_material.DialogContent, {
                 sx: {
                     width: '500px'
                 },
-                children: `Are you sure you want to delete ${deletePanelDialog.panelName} from ${(ref = layouts[deletePanelDialog.layoutItem.groupIndex]) === null || ref === void 0 ? void 0 : ref.title}? This action cannot be undone.`
+                children: [
+                    "Are you sure you want to delete ",
+                    deletePanelDialog.panelName,
+                    " from ",
+                    deletePanelDialog.panelGroupName,
+                    "? This action cannot be undone."
+                ]
             }),
             /*#__PURE__*/ (0, _jsxRuntime.jsxs)(_material.DialogActions, {
                 children: [

package/dist/cjs/components/Panel/Panel.js

@@ -25,7 +25,7 @@ const _reactIntersectionObserver = require("react-intersection-observer");
 const _components = require("@perses-dev/components");
 const _material = require("@mui/material");
 const _informationOutline = /*#__PURE__*/ _interopRequireDefault(require("mdi-material-ui/InformationOutline"));
-const _pencil = /*#__PURE__*/ _interopRequireDefault(require("mdi-material-ui/Pencil"));
+const _pencilOutline = /*#__PURE__*/ _interopRequireDefault(require("mdi-material-ui/PencilOutline"));
 const _deleteOutline = /*#__PURE__*/ _interopRequireDefault(require("mdi-material-ui/DeleteOutline"));
 const _dragVertical = /*#__PURE__*/ _interopRequireDefault(require("mdi-material-ui/DragVertical"));
 const _context = require("../../context");
@@ -36,7 +36,7 @@ function _interopRequireDefault(obj) {
     };
 }
 function Panel(props) {
-    const { definition , groupIndex , itemIndex , ...others } = props;
+    const { definition , panelGroupItemId , ...others } = props;
     const [contentElement, setContentElement] = (0, _react.useState)(null);
     const [isHovered, setIsHovered] = (0, _react.useState)(false);
     const { width , height  } = (0, _useResizeObserver.default)({
@@ -60,13 +60,7 @@ function Panel(props) {
     // TODO: adjust padding for small panels, consistent way to determine isLargePanel here and in StatChart
     const panelPadding = 1.5;
     const { isEditMode  } = (0, _context.useEditMode)();
-    const { editPanel , openDeletePanelDialog  } = (0, _context.usePanels)();
-    const handleEditButtonClick = ()=>{
-        editPanel({
-            groupIndex,
-            itemIndex
-        });
-    };
+    const { openEditPanel , openDeletePanelDialog  } = (0, _context.usePanelActions)(panelGroupItemId);
     return /*#__PURE__*/ (0, _jsxRuntime.jsxs)(_material.Card, {
         ref: ref,
         sx: {
@@ -86,13 +80,11 @@ function Panel(props) {
                     sx: {
                         display: 'flex',
                         alignItems: 'center',
-                        minHeight: '24px'
+                        minHeight: '32px'
                     },
                     children: [
                         /*#__PURE__*/ (0, _jsxRuntime.jsx)(_material.Typography, {
-                            component: "h2",
-                            variant: "body2",
-                            fontWeight: (theme)=>theme.typography.fontWeightMedium,
+                            variant: "subtitle1",
                             whiteSpace: "nowrap",
                             overflow: "hidden",
                             textOverflow: "ellipsis",
@@ -112,9 +104,9 @@ function Panel(props) {
                                     children: /*#__PURE__*/ (0, _jsxRuntime.jsx)(_informationOutline.default, {
                                         "aria-describedby": "info-tooltip",
                                         "aria-hidden": false,
-                                        fontSize: "small",
                                         sx: {
-                                            cursor: 'pointer'
+                                            cursor: 'pointer',
+                                            color: (theme)=>theme.palette.grey[700]
                                         }
                                     })
                                 }),
@@ -126,16 +118,13 @@ function Panel(props) {
                                         /*#__PURE__*/ (0, _jsxRuntime.jsx)(IconButton, {
                                             "aria-label": "edit panel",
                                             size: "small",
-                                            onClick: handleEditButtonClick,
-                                            children: /*#__PURE__*/ (0, _jsxRuntime.jsx)(_pencil.default, {})
+                                            onClick: openEditPanel,
+                                            children: /*#__PURE__*/ (0, _jsxRuntime.jsx)(_pencilOutline.default, {})
                                         }),
                                         /*#__PURE__*/ (0, _jsxRuntime.jsx)(IconButton, {
                                             "aria-label": "delete panel",
                                             size: "small",
-                                            onClick: ()=>openDeletePanelDialog({
-                                                    groupIndex,
-                                                    itemIndex
-                                                }),
+                                            onClick: openDeletePanelDialog,
                                             children: /*#__PURE__*/ (0, _jsxRuntime.jsx)(_deleteOutline.default, {})
                                         }),
                                         /*#__PURE__*/ (0, _jsxRuntime.jsx)(IconButton, {
@@ -156,7 +145,8 @@ function Panel(props) {
                 }),
                 sx: {
                     display: 'block',
-                    padding: (theme)=>theme.spacing(1, panelPadding),
+                    paddingX: (theme)=>theme.spacing(panelPadding),
+                    paddingY: '4px',
                     borderBottom: (theme)=>`solid 1px ${theme.palette.divider}`
                 }
             }),

package/dist/cjs/components/Panel/Panel.test.js

@@ -16,7 +16,6 @@ Object.defineProperty(exports, "__esModule", {
 });
 const _jsxRuntime = require("react/jsx-runtime");
 const _pluginSystem = require("@perses-dev/plugin-system");
-require("intersection-observer");
 const _react = require("@testing-library/react");
 const _userEvent = /*#__PURE__*/ _interopRequireDefault(require("@testing-library/user-event"));
 const _test = require("../../test");
@@ -44,8 +43,11 @@ describe('Panel', ()=>{
                     }
                 }
             },
-            groupIndex: 0,
-            itemIndex: 0
+            // TODO: This is coupled to ID generation which is not good and the tests will probably fail
+            panelGroupItemId: {
+                panelGroupId: 0,
+                panelGroupItemLayoutId: ''
+            }
         };
     };
     // Helper to render the panel with some context set

package/dist/cjs/components/Panel/index.js

@@ -15,6 +15,7 @@ Object.defineProperty(exports, "__esModule", {
     value: true
 });
 _exportStar(require("./Panel"), exports);
+_exportStar(require("./DeletePanelDialog"), exports);
 function _exportStar(from, to) {
     Object.keys(from).forEach(function(k) {
         if (k !== "default" && !Object.prototype.hasOwnProperty.call(to, k)) Object.defineProperty(to, k, {

package/dist/cjs/components/PanelDrawer/PanelDrawer.js

@@ -25,7 +25,7 @@ const _components = require("@perses-dev/components");
 const _context = require("../../context");
 const _panelEditorForm = require("./PanelEditorForm");
 const PanelDrawer = ()=>{
-    const { panelEditor  } = (0, _context.usePanels)();
+    const panelEditor = (0, _context.usePanelEditor)();
     // When the user clicks close, start closing but don't call the store yet to keep values stable during animtation
     const [isClosing, setIsClosing] = (0, _react.useState)(false);
     const handleClose = ()=>setIsClosing(true);

package/dist/cjs/components/PanelDrawer/PanelDrawer.test.js

@@ -53,8 +53,8 @@ describe('Panel Drawer', ()=>{
     };
     it('should add new panel', async ()=>{
         const storeApi = renderPanelDrawer();
-        // Open the drawer for a new panel (i.e. no panel key)
-        (0, _testUtils.act)(()=>storeApi.getState().addPanel(0));
+        // Open the drawer for a new panel
+        (0, _testUtils.act)(()=>storeApi.getState().openAddPanel());
         const nameInput = await _react.screen.findByLabelText(/Name/);
         _userEvent.default.type(nameInput, 'New Panel');
         _userEvent.default.click(_react.screen.getByText('Add'));
@@ -79,9 +79,17 @@ describe('Panel Drawer', ()=>{
     it('should edit an existing panel', async ()=>{
         const storeApi = renderPanelDrawer();
         // Open the drawer for an existing panel
-        (0, _testUtils.act)(()=>storeApi.getState().editPanel({
-                groupIndex: 0,
-                itemIndex: 0
+        const group = Object.values(storeApi.getState().panelGroups).find((group)=>group.title === 'CPU Stats');
+        if (group === undefined) {
+            throw new Error('Test group not found');
+        }
+        const layout = Object.entries(group.itemPanelKeys).find(([, panelKey])=>panelKey === 'cpu');
+        if (layout === undefined) {
+            throw new Error('Test panel not found');
+        }
+        (0, _testUtils.act)(()=>storeApi.getState().openEditPanel({
+                panelGroupId: group.id,
+                panelGroupItemLayoutId: layout[0]
             }));
         const nameInput = await _react.screen.findByLabelText(/Name/);
         _userEvent.default.clear(nameInput);

package/dist/cjs/components/PanelDrawer/PanelEditorForm.js

@@ -34,10 +34,10 @@ const _panelPreview = require("./PanelPreview");
 function PanelEditorForm(props) {
     var ref;
     const { initialValues , onSubmit  } = props;
-    const { layouts  } = (0, _context.useLayouts)();
+    const panelGroups = (0, _context.useListPanelGroups)();
     const [name, setName] = (0, _react.useState)(initialValues.name);
     const [description, setDescription] = (0, _react.useState)(initialValues.description);
-    const [groupIndex, setGroupIndex] = (0, _react.useState)(initialValues.groupIndex);
+    const [groupId, setGroupId] = (0, _react.useState)(initialValues.groupId);
     const [kind, setKind] = (0, _react.useState)(initialValues.kind);
     const [spec, setSpec] = (0, _react.useState)(initialValues.spec);
     // Use common plugin editor logic even though we've split the inputs up in this form
@@ -58,35 +58,38 @@ function PanelEditorForm(props) {
         if (typeof value === 'string') {
             return;
         }
-        setGroupIndex(value);
+        setGroupId(value);
     };
     const handleSubmit = (e)=>{
         e.preventDefault();
-        if (spec === undefined) {
-            throw new Error('Cannot submit without a plugin spec');
-        }
         const values = {
             name,
             description,
-            groupIndex,
+            groupId,
             kind,
             spec
         };
         onSubmit(values);
     };
     var _title, ref1;
-    return /*#__PURE__*/ (0, _jsxRuntime.jsx)("form", {
+    return(// Grid maxHeight allows user to scroll inside Drawer to see all content
+    /*#__PURE__*/ (0, _jsxRuntime.jsx)("form", {
         id: panelEditorFormId,
         onSubmit: handleSubmit,
         children: /*#__PURE__*/ (0, _jsxRuntime.jsxs)(_material.Grid, {
             container: true,
             spacing: 2,
+            sx: {
+                overflowY: 'scroll',
+                maxHeight: '90vh'
+            },
             children: [
                 /*#__PURE__*/ (0, _jsxRuntime.jsx)(_material.Grid, {
                     item: true,
                     xs: 8,
                     children: /*#__PURE__*/ (0, _jsxRuntime.jsx)(_material.TextField, {
                         required: true,
+                        fullWidth: true,
                         label: "Name",
                         value: name,
                         variant: "outlined",
@@ -97,6 +100,7 @@ function PanelEditorForm(props) {
                     item: true,
                     xs: 4,
                     children: /*#__PURE__*/ (0, _jsxRuntime.jsxs)(_material.FormControl, {
+                        fullWidth: true,
                         children: [
                             /*#__PURE__*/ (0, _jsxRuntime.jsx)(_material.InputLabel, {
                                 id: "select-group",
@@ -106,12 +110,12 @@ function PanelEditorForm(props) {
                                 required: true,
                                 labelId: "select-group",
                                 label: "Group",
-                                value: groupIndex !== null && groupIndex !== void 0 ? groupIndex : 0,
+                                value: groupId,
                                 onChange: handleGroupChange,
-                                children: layouts.map((layout, index)=>/*#__PURE__*/ (0, _jsxRuntime.jsx)(_material.MenuItem, {
-                                        value: index,
-                                        children: (_title = layout.title) !== null && _title !== void 0 ? _title : `Group ${index + 1}`
-                                    }, index))
+                                children: panelGroups.map((panelGroup, index)=>/*#__PURE__*/ (0, _jsxRuntime.jsx)(_material.MenuItem, {
+                                        value: panelGroup.id,
+                                        children: (_title = panelGroup.title) !== null && _title !== void 0 ? _title : `Group ${index + 1}`
+                                    }, panelGroup.id))
                             })
                         ]
                     })
@@ -120,6 +124,7 @@ function PanelEditorForm(props) {
                     item: true,
                     xs: 8,
                     children: /*#__PURE__*/ (0, _jsxRuntime.jsx)(_material.TextField, {
+                        fullWidth: true,
                         label: "Description",
                         value: description,
                         variant: "outlined",
@@ -131,6 +136,7 @@ function PanelEditorForm(props) {
                     xs: 4,
                     children: [
                         /*#__PURE__*/ (0, _jsxRuntime.jsxs)(_material.FormControl, {
+                            fullWidth: true,
                             disabled: pluginEditor.isLoading,
                             error: pluginEditor.error !== null,
                             children: [
@@ -169,7 +175,7 @@ function PanelEditorForm(props) {
                                 name: name,
                                 description: description,
                                 spec: spec,
-                                groupIndex: groupIndex
+                                groupId: groupId
                             })
                         })
                     ]
@@ -189,6 +195,6 @@ function PanelEditorForm(props) {
                 })
             ]
         })
-    });
+    }));
 }
 const panelEditorFormId = 'panel-editor-form';

package/dist/cjs/components/PanelDrawer/PanelPreview.js

@@ -21,7 +21,7 @@ Object.defineProperty(exports, "PanelPreview", {
 const _jsxRuntime = require("react/jsx-runtime");
 const _material = require("@mui/material");
 const _panel = require("../Panel");
-function PanelPreview({ name , description , kind , spec , groupIndex  }) {
+function PanelPreview({ name , description , kind , spec , groupId  }) {
     const previewValues = {
         definition: {
             kind: 'Panel',
@@ -36,9 +36,15 @@ function PanelPreview({ name , description , kind , spec , groupIndex  }) {
                 }
             }
         },
-        groupIndex,
-        itemIndex: 0
+        // TODO: this shouldn't be necessary for preview
+        panelGroupItemId: {
+            panelGroupId: groupId,
+            panelGroupItemLayoutId: ''
+        }
     };
+    if (!kind) {
+        return null;
+    }
     return /*#__PURE__*/ (0, _jsxRuntime.jsx)(_material.Box, {
         height: 300,
         children: /*#__PURE__*/ (0, _jsxRuntime.jsx)(_panel.Panel, {