Merge pull request #53 from abbyhu2000/revert-grid-item

revert back to grid, reference too much of issue without hooks

src/plugins/dashboard/public/application/components/dashboard_editor.tsx

@@ -39,15 +39,6 @@ export const DashboardEditor = () => {
     dashboard
   );
 
-  // const { dashboardContainer } = useDashboardContainer(
-  //   services,
-  //   isChromeVisible,
-  //   eventEmitter,
-  //   dashboard,
-  //   savedDashboardInstance,
-  //   appState
-  // );
-
   const { isEmbeddableRendered, currentAppState } = useEditorUpdates(
     eventEmitter,
     dashboard,

src/plugins/dashboard/public/application/embeddable/grid/dashboard_grid.tsx

@@ -47,7 +47,6 @@ import { DashboardPanelState } from '../types';
 import { withOpenSearchDashboards } from '../../../../../opensearch_dashboards_react/public';
 import { DashboardContainerInput } from '../dashboard_container';
 import { DashboardContainer, DashboardReactContextValue } from '../dashboard_container';
-import { DashboardGridItem } from './dashboard_grid_item';
 
 let lastValidGridSize = 0;
 
@@ -263,18 +262,34 @@ class DashboardGridUi extends React.Component<DashboardGridProps, State> {
       }
     });
 
-    return _.map(panelsInOrder, ({ explicitInput, type }) => (
-      <DashboardGridItem
-        expandedPanelId={expandedPanelId}
-        focusedPanelIndex={focusedPanelIndex}
-        id={explicitInput.id}
-        key={explicitInput.id}
-        type={type}
-        gridItems={this.gridItems}
-        container={this.props.container}
-        PanelComponent={this.props.PanelComponent}
-      />
-    ));
+    return _.map(panelsInOrder, (panel) => {
+      const expandPanel =
+        expandedPanelId !== undefined && expandedPanelId === panel.explicitInput.id;
+      const hidePanel = expandedPanelId !== undefined && expandedPanelId !== panel.explicitInput.id;
+      const classes = classNames({
+        // eslint-disable-next-line @typescript-eslint/naming-convention
+        'dshDashboardGrid__item--expanded': expandPanel,
+        // eslint-disable-next-line @typescript-eslint/naming-convention
+        'dshDashboardGrid__item--hidden': hidePanel,
+      });
+      return (
+        <div
+          style={{ zIndex: focusedPanelIndex === panel.explicitInput.id ? 2 : 'auto' }}
+          className={classes}
+          key={panel.explicitInput.id}
+          data-test-subj="dashboardPanel"
+          ref={(reactGridItem) => {
+            this.gridItems[panel.explicitInput.id] = reactGridItem;
+          }}
+        >
+          <EmbeddableChildPanel
+            embeddableId={panel.explicitInput.id}
+            container={this.props.container}
+            PanelComponent={this.props.PanelComponent}
+          />
+        </div>
+      );
+    });
   }
 
   public render() {