[DataGrid] Log error if rowCount is used with client-side pagination

[michelengelen]

• I have followed (at least) the PR section of the contributing guide.

Added a warning that the rowCount prop has no effect when using paginationMode = client.

fixes #7303

[mui-bot]

Deploy preview: https://deploy-preview-12448--material-ui-x.netlify.app/

Updated pages:

• docs/data/data-grid/pagination/pagination.md

Generated by 🚫 dangerJS against f645c4d

[cherniavskii]

Should we also add a runtime warning in dev mode?
Something like:

mui-x/packages/x-data-grid/src/internals/utils/propValidation.ts

Line 5 in a518d0a

export const propValidatorsDataGrid: PropValidator<DataGridProcessedProps>[] = [

[MBilalShafi]

Thank you 👍

[michelengelen]

@LukasTy the tests failing seem unrelated to this PR. Could you have a look since the ones failing are for DesktopDateTimeRangePicker and MobileDateTimeRangePicker?

  1) <DesktopDateTimeRangePicker /> - Describes
       Value API
         Picker open / close lifecycle
           should not call onClose or onAccept when selecting a date and `props.closeOnSelect` is false:
     Error: Timeout of 5000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves. (/tmp/mui/packages/x-date-pickers-pro/src/DesktopDateTimeRangePicker/tests/describes.DesktopDateTimeRangePicker.test.tsx)
      at processImmediate (node:internal/timers:476:21)

  2) <MobileDateTimeRangePicker /> - Describes
       Value API
         Picker open / close lifecycle
           should not call onClose or onAccept when selecting a date and `props.closeOnSelect` is false:
     Error: Timeout of 5000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves. (/tmp/mui/packages/x-date-pickers-pro/src/MobileDateTimeRangePicker/tests/describes.MobileDateTimeRangePicker.test.tsx)
      at processImmediate (node:internal/timers:476:21)

[LukasTy]

@michelengelen Could you first sync with master and see if it helps? 🤔
We had an actual problem with DesktopDateTimeRangePicker and MobileDateTimeRangePicker seems to be flaky. 🙈

[lauri865]

Is there a deeper reason for this error message or just to warn the developer @michelengelen?
We don't use server side fetching, but use filtering outside of the Datagrid. So, there's a subset of the total rows supplied to the grid in some occasions. We use rowCount to display the total number of rows in the footer.

Now we get an annoying error message. We know it doesn't have any "effect", but we need it for display reasons anyways. Unless there's an actual error, maybe warning would be more appropriate (and only in dev), otherwise, it's unnecessary code that gets bundled into production it seems?