fix(UI): refactor responsive grid layout #closes2114

Canadian-Geospatial-Platform · May 16, 2024 · 69203ae · 69203ae
1 parent fdcf58d
commit 69203ae
Showing 1 changed file with 18 additions and 16 deletions.
diff --git a/packages/geoview-core/src/core/components/common/responsive-grid.tsx b/packages/geoview-core/src/core/components/common/responsive-grid.tsx
@@ -56,7 +56,15 @@ const getLeftPanelSize = (fullWidth: boolean, isRightPanelVisible: boolean, isEn
  */
 const ResponsiveGridLeftPanel = forwardRef(
   (
-    { children, className, isRightPanelVisible = false, sxProps = {}, isEnlarged, fullWidth = false, ...rest }: ResponsiveGridPanelProps,
+    {
+      children,
+      className = '',
+      isRightPanelVisible = false,
+      sxProps = {},
+      isEnlarged,
+      fullWidth = false,
+      ...rest
+    }: ResponsiveGridPanelProps,
     ref
   ) => {
     const theme = useTheme();
@@ -111,7 +119,15 @@ const getRightPanelSize = (fullWidth: boolean, isRightPanelVisible: boolean, isE
  */
 const ResponsiveGridRightPanel = forwardRef(
   (
-    { children, className, isRightPanelVisible = false, sxProps = {}, isEnlarged, fullWidth = false, ...rest }: ResponsiveGridPanelProps,
+    {
+      children,
+      className = '',
+      isRightPanelVisible = false,
+      sxProps = {},
+      isEnlarged,
+      fullWidth = false,
+      ...rest
+    }: ResponsiveGridPanelProps,
     ref
   ) => {
     const theme = useTheme();
@@ -143,17 +159,3 @@ export const ResponsiveGrid = {
   Left: ResponsiveGridLeftPanel,
   Right: ResponsiveGridRightPanel,
 };
-
-// TODO: Refactor - Remove defaultProps as it's no longer a good practice
-ResponsiveGridLeftPanel.defaultProps = {
-  sxProps: undefined,
-  fullWidth: false,
-  className: '',
-};
-
-// TODO: Refactor - Remove defaultProps as it's no longer a good practice
-ResponsiveGridRightPanel.defaultProps = {
-  sxProps: undefined,
-  fullWidth: false,
-  className: '',
-};