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] Fix description for "Striped rows" example #5432

Merged
merged 1 commit into from
Jul 12, 2022

Conversation

lindapaiste
Copy link
Contributor

This PR fixes a mistake introduced in #3882

Mistake

The description for the "Custom theme" example contains text that is meant for the "Striped rows" example:
image

While the "Striped rows" example description has a typo ("stripped" instead of "striped"):
image

Source

The mistake happened in this commit.
image

Fix

@mui-bot
Copy link

mui-bot commented Jul 8, 2022

These are the results for the performance tests:

Test case Unit Min Max Median Mean σ
Filter 100k rows ms 243.8 404.4 317.8 335.4 61.382
Sort 100k rows ms 444.7 929 713.9 698.7 162.081
Select 100k rows ms 144.4 200.2 178.6 174.56 19.63
Deselect 100k rows ms 109.7 280.1 201.6 182.88 63.956

Generated by 🚫 dangerJS against 753fad3

@m4theushw m4theushw changed the title [docs] fix description for "Striped rows" example [docs] Fix description for "Striped rows" example Jul 12, 2022
@m4theushw m4theushw added the docs Improvements or additions to the documentation label Jul 12, 2022
Copy link
Member

@m4theushw m4theushw left a comment

Choose a reason for hiding this comment

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

Thanks!

@m4theushw m4theushw merged commit b071648 into mui:master Jul 12, 2022
joserodolfofreitas pushed a commit to joserodolfofreitas/mui-x that referenced this pull request Jul 15, 2022
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.

3 participants