-
-
Notifications
You must be signed in to change notification settings - Fork 32.5k
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
[website] Mark TreeView and Charts as stable #39975
Conversation
| [Tree View - Multiselect](/x/react-tree-view/#multi-selection) | MUI X | 🧪 | | ||
| Tree View - Virtualization | MUI X | ⏳ | | ||
| Window Splitter | MUI X | ⏳ | | ||
| Name | Product | Status | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not sure why the entire table got diffed.
I just removed the nested tree view items and marked the tree view as stable.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it's because [Tree View - Multiselect](/x/react-tree-view/#multi-selection)
was removed, reducing the width of the first column.
It looks like GitHub diff UI is broken, it's failing to show the diff correctly.
Netlify deploy previewBundle size report |
Using D3.js for data manipulation and SVG for rendering, Charts are now available on MUI | ||
X! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it would be best to use a non-breaking space here to avoid:

I'm not sure GitHub suggestion UI is handling non-breaking space correctly anymore, I have tried this but it doesn't seem to work:
Using D3.js for data manipulation and SVG for rendering, Charts are now available on MUI | |
X! | |
Using D3.js for data manipulation and SVG for rendering, Charts are now available on MUI X! |
It might need to be done with a direct commit.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good for me.
Don't bother too much with breaking space, because the text is removed in #39832
@danilo-leal I you get breaking changes on your PR, it' might be because of that one
@alexfauquette Oh, sweet, thanks for the heads-up! The marketing page demos will probably be re-worked on my PR that you linked but all good with the roadmap page changes here 🤙 |
Preview: https://deploy-preview-39975--material-ui.netlify.app/x/