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 toggle button bug in demos in Custom Components page #6913

Merged
merged 1 commit into from
Nov 21, 2022

Conversation

01zulfi
Copy link
Contributor

@01zulfi 01zulfi commented Nov 18, 2022

Related #6896

  • Clicking on the toggle button twice makes the content disappear
  • This PR fixes that in two demos in the page
  • This PR makes sure to make relevant changes for both the .js and .tsx files

The demos in question: https://next.mui.com/x/react-date-pickers/custom-components/#component-3

* Clicking on a toggle button twice makes the content disappear
* This commit fixes that in two demos in the page
* This commit makes sure to make relevant changes for both the .js and .tsx files
@mui-bot
Copy link

mui-bot commented Nov 18, 2022

Messages
📖 Netlify deploy preview: https://deploy-preview-6913--material-ui-x.netlify.app/

These are the results for the performance tests:

Test case Unit Min Max Median Mean σ
Filter 100k rows ms 704.5 1,193.8 704.5 941.42 179.03
Sort 100k rows ms 778.7 1,284.1 1,284.1 1,011.58 161.957
Select 100k rows ms 239.9 363.4 264.7 284.1 44.078
Deselect 100k rows ms 160.2 312.3 216.1 231.98 50.705

Generated by 🚫 dangerJS against fbc6713

@LukasTy LukasTy added docs Improvements or additions to the documentation component: pickers This is the name of the generic UI component, not the React module! labels Nov 18, 2022
Copy link
Member

@flaviendelangle flaviendelangle left a comment

Choose a reason for hiding this comment

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

Thanks again for your contribution !

This PR makes sure to make relevant changes for both the .js and .tsx files

I don't know if you made the changes on the .js file by hand
But just in case, you can automatically generate it with yarn docs:typescript:formatted

@flaviendelangle flaviendelangle merged commit f69ef74 into mui:next Nov 21, 2022
@01zulfi
Copy link
Contributor Author

01zulfi commented Nov 21, 2022

@flaviendelangle I totally made all those changes manually 😁

Thanks for the tip ❤️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: pickers This is the name of the generic UI component, not the React module! docs Improvements or additions to the documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants