Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[data grid] ResizeObserver loop error when using getRowHeight = {() = 'auto'} #16197

Open
kseileraltus opened this issue Jan 15, 2025 · 4 comments
Labels
component: data grid This is the name of the generic UI component, not the React module! status: waiting for author Issue with insufficient information support: commercial Support request from paid users support: pro standard Support request from a Pro standard plan user. https://mui.com/legal/technical-support-sla/

Comments

@kseileraltus
Copy link

kseileraltus commented Jan 15, 2025

The problem in depth

Apologies, I wasn't sure if this is a bug with MUI or a problem with my environment so I've filed it here.

I am only able to see this on my local instance of the application, but when I try to use 'getRowHeight={() => 'auto'}' as shown in the documentation I run into a ResizeObserver error. This happens even with the most basic DataGrid as seen in the screenshots below.

Image

Image

Image

As you can see, the table thats causing this error is very basic with all static values. If I replace 'auto' with a fixed value the issue goes away. I noticed some recent fixes in related areas here and here, but not sure how to proceed. Any help would be greatly appreciated.

Your environment

`npx @mui/envinfo`
  System:
    OS: macOS 14.6.1
  Binaries:
    Node: 18.17.1 - ~/.nvm/versions/node/v18.17.1/bin/node
    npm: 9.6.7 - ~/.nvm/versions/node/v18.17.1/bin/npm
    pnpm: Not Found
  Browsers:
    Chrome: 131.0.6778.265
    Edge: Not Found
    Safari: 17.6
  npmPackages:
    @emotion/react: ^11.11.1 => 11.14.0 
    @emotion/styled: ^11.11.0 => 11.14.0 
    @mui/core-downloads-tracker:  5.16.14 
    @mui/material: ^5.14.4 => 5.16.14 
    @mui/private-theming:  5.16.14 
    @mui/styled-engine:  5.16.14 
    @mui/system:  5.16.14 
    @mui/types:  7.2.21 
    @mui/utils:  6.4.0 
    @mui/x-data-grid:  7.23.6 
    @mui/x-data-grid-pro: ^7.23.6 => 7.23.6 
    @mui/x-internals:  7.23.6 
    @mui/x-license: ^7.1.1 => 7.23.6 
    @types/react: ^18.0.28 => 18.3.18 
    react: 17.0.2 => 17.0.2 
    react-dom: 17.0.2 => 17.0.2 
    styled-components: ^5.1.1 => 5.3.11 
    typescript: ^5.4.2 => 5.7.3 

Search keywords: ResizeObserver getRowHeight auto

Order ID: 88197

@kseileraltus kseileraltus added status: waiting for maintainer These issues haven't been looked at yet by a maintainer support: commercial Support request from paid users labels Jan 15, 2025
@github-actions github-actions bot added component: data grid This is the name of the generic UI component, not the React module! support: pro standard Support request from a Pro standard plan user. https://mui.com/legal/technical-support-sla/ labels Jan 15, 2025
@michelengelen
Copy link
Member

Hey @kseileraltus ... I am not able to reproduce this. Is there anything specific you did to your environment that would lead to your assumption it has something to do with that?

@michelengelen michelengelen added status: waiting for author Issue with insufficient information and removed status: waiting for maintainer These issues haven't been looked at yet by a maintainer labels Jan 16, 2025
@michelengelen michelengelen changed the title [question] Seeing a 'ResizeObserver loop' error when using getRowHeight = {() = 'auto'} [data grid] ResizeObserver loop error when using getRowHeight = {() = 'auto'} Jan 16, 2025
@michelengelen michelengelen changed the title [data grid] ResizeObserver loop error when using getRowHeight = {() = 'auto'} [data grid] ResizeObserver loop error when using getRowHeight = {() = 'auto'} Jan 16, 2025
@kseileraltus
Copy link
Author

Hi @michelengelen No the environment isn't doing specific 🤔

@github-actions github-actions bot added status: waiting for maintainer These issues haven't been looked at yet by a maintainer and removed status: waiting for author Issue with insufficient information labels Jan 16, 2025
@romgrk
Copy link
Contributor

romgrk commented Jan 17, 2025

Unless the error can consistently be reproduced (edit: in production, ideally), it's usually not worth investigating: https://trackjs.com/javascript-errors/resizeobserver-loop-completed-with-undelivered-notifications/

The error messages from that API are overly zealous considering how browsers implement it.

@michelengelen
Copy link
Member

@kseileraltus can you consistently reproduce this?

@michelengelen michelengelen added status: waiting for author Issue with insufficient information and removed status: waiting for maintainer These issues haven't been looked at yet by a maintainer labels Jan 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: data grid This is the name of the generic UI component, not the React module! status: waiting for author Issue with insufficient information support: commercial Support request from paid users support: pro standard Support request from a Pro standard plan user. https://mui.com/legal/technical-support-sla/
Projects
None yet
Development

No branches or pull requests

3 participants