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

[docs] Replace custom notes and warning with callouts #5008

Merged
merged 11 commits into from
May 31, 2022

Conversation

flaviendelangle
Copy link
Member

@flaviendelangle flaviendelangle commented May 25, 2022

@mui-bot
Copy link

mui-bot commented May 25, 2022

These are the results for the performance tests:

Test case Unit Min Max Median Mean σ
Filter 100k rows ms 349.2 678.8 457.3 472.5 118.071
Sort 100k rows ms 623.9 1,053.3 623.9 866.24 152.836
Select 100k rows ms 144.8 281 194.1 199.94 46.359
Deselect 100k rows ms 112.7 327.8 203.3 196.52 78.406

Generated by 🚫 dangerJS against 812efbf

docs/data/data-grid/column-definition/column-definition.md Outdated Show resolved Hide resolved
docs/data/data-grid/column-definition/column-definition.md Outdated Show resolved Hide resolved
docs/data/data-grid/column-spanning/column-spanning.md Outdated Show resolved Hide resolved
});
};
```

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change

```tsx
<DataGridPro rowThreshold={0} />
```

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change

docs/data/data-grid/pagination/pagination.md Outdated Show resolved Hide resolved
```tsx
<DataGrid getRowId={(row) => row.internalId} />
```

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There're other empty lines before closing the callout.

Suggested change

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it is our prettier config which always adds an empty line after a code block

@oliviertassinari oliviertassinari added the docs Improvements or additions to the documentation label May 25, 2022
@github-actions
Copy link

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@github-actions github-actions bot added the PR: out-of-date The pull request has merge conflicts and can't be merged label May 31, 2022
@github-actions github-actions bot removed the PR: out-of-date The pull request has merge conflicts and can't be merged label May 31, 2022
@flaviendelangle flaviendelangle merged commit 1cfef48 into mui:master May 31, 2022
@flaviendelangle flaviendelangle deleted the callouts branch May 31, 2022 14:21
alexfauquette pushed a commit to alexfauquette/mui-x that referenced this pull request Aug 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Improvements or additions to the documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants