From 3f45f07afb2c18c5c2ff9a16e3c48f6ce0d70dda Mon Sep 17 00:00:00 2001 From: Matthew Runyon Date: Mon, 10 Jun 2024 11:30:59 -0500 Subject: [PATCH] fix: Embed-widget with multiple panels not showing panel headers (#2064) This should fix embed-widget not showing the panel headers when there are multiple panels in a widget, but it's not a dashboard. The issue is embed-widget is loading the layout from `dashboardData` and the `DEFAULT_DASHBOARD_ID` stores its data in `workspaceData` while all other IDs use `dashboardData`. Tested with widget with just 1 panel, widget with multiple panels, and widget that is a dashboard. --- packages/embed-widget/src/App.tsx | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/packages/embed-widget/src/App.tsx b/packages/embed-widget/src/App.tsx index a427ee19a..ab7931e6d 100644 --- a/packages/embed-widget/src/App.tsx +++ b/packages/embed-widget/src/App.tsx @@ -18,7 +18,6 @@ import { getAllDashboardsData, listenForCreateDashboard, CreateDashboardPayload, - DEFAULT_DASHBOARD_ID, setDashboardPluginData, stopListenForCreateDashboard, } from '@deephaven/dashboard'; @@ -98,7 +97,7 @@ function App(): JSX.Element { }, [dispatch, user]); const [goldenLayout, setGoldenLayout] = useState(null); - const [dashboardId, setDashboardId] = useState(DEFAULT_DASHBOARD_ID); + const [dashboardId, setDashboardId] = useState('default-embed-widget'); // Can't be DEFAULT_DASHBOARD_ID because its dashboard layout is not stored in dashboardData const handleGoldenLayoutChange = useCallback( (newLayout: GoldenLayout) => {