[data grid] DataGrid inside Dialog doesn't show the full column list about mui-x HOT 6 OPEN

@michelengelen

The flip was disabled in #837 (comment). I believe the goal was to avoid this effect:

How about doing this instead?

{
  name: 'flip',
  enabled: true,
  options: {
    rootBoundary: 'document'
  }
},

from mui-x.

Hey @simbahandiane the sandbox you shared is not public.
Could you change that so we can have a look?
Thanks! 🙇🏼

from mui-x.

@michelengelen Can you try this link and let me know if you can access it now?

from mui-x.

Yes, it works.
I did check the underlying code for the panel and I think we could just change the default behavior.

diff --git a/packages/x-data-grid/src/components/panel/GridPanel.tsx b/packages/x-data-grid/src/components/panel/GridPanel.tsx
index a82b53b37..6109783f4 100644
--- a/packages/x-data-grid/src/components/panel/GridPanel.tsx
+++ b/packages/x-data-grid/src/components/panel/GridPanel.tsx
@@ -79,7 +79,10 @@ const GridPanel = React.forwardRef<HTMLDivElement, GridPanelProps>((props, ref)
     () => [
       {
         name: 'flip',
-        enabled: false,
+        enabled: true,
+        options: {
+          rootBoundary: 'document'
+        },
       },
       {
         name: 'isPlaced',

Should we just make this change @cherniavskii ?

from mui-x.

@michelengelen

The flip was disabled in #837 (comment). I believe the goal was to avoid this effect:

Screen.Recording.2024-06-14.at.17.24.29.mov
How about doing this instead?

{
  name: 'flip',
  enabled: true,
  options: {
    rootBoundary: 'document'
  }
},

Yes, sounds good.
I have updated my diff.
Lets label this a good first issue

from mui-x.

@michelengelen @cherniavskii We are currently in version 6. Is there any way that this fix can also be published on version 6? If it can't be, is there a work around that we can do for now?

from mui-x.