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.

Sign up for GitHub

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

Jump to bottom

[tree view] Fix demo overflow #11976

Closed
oliviertassinari opened this issue Feb 7, 2024 · 1 comment · Fixed by #11979
Closed

[tree view] Fix demo overflow #11976

oliviertassinari opened this issue Feb 7, 2024 · 1 comment · Fixed by #11979
Assignees
@noraleonte
Labels
bug 🐛 Something doesn't work component: tree view TreeView, TreeItem. This is the name of the generic UI component, not the React module! docs Improvements or additions to the documentation

Comments

@oliviertassinari
Copy link
Member

oliviertassinari commented Feb 7, 2024 •
edited by github-actions bot
Loading

Steps to reproduce

There are a few demo heights to update, e.g.

https://deploy-preview-11529--material-ui-x.netlify.app/x/react-tree-view/simple-tree-view/items/#basics

The values were set for denser tree view item heights. In some cases, we would probably want a overflow: 'auto' or a dynamic height like in https://deploy-preview-11529--material-ui-x.netlify.app/x/react-tree-view/simple-tree-view/customization/#custom-icons

Current behavior

No response

Expected behavior

No response

Context

No response

Your environment

npx @mui/envinfo 
  Don't forget to mention which browser you used.
  Output from `npx @mui/envinfo` goes here.

Search keywords: tree view overflow
@oliviertassinari oliviertassinari added bug 🐛 Something doesn't work status: waiting for maintainer These issues haven't been looked at yet by a maintainer component: tree view TreeView, TreeItem. This is the name of the generic UI component, not the React module! labels Feb 7, 2024
@oliviertassinari oliviertassinari mentioned this issue Feb 7, 2024
Merged
1 task
@noraleonte noraleonte self-assigned this Feb 7, 2024
@noraleonte noraleonte added docs Improvements or additions to the documentation and removed status: waiting for maintainer These issues haven't been looked at yet by a maintainer labels Feb 7, 2024
@danilo-leal danilo-leal mentioned this issue Feb 7, 2024
Merged
1 task
@github-actions GitHub Actions
Copy link

⚠️ This issue has been closed.
If you have a similar problem, please open a new issue and provide details about your specific problem.
If you can provide additional information related to this topic that could help future readers, please feel free to leave a comment.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@noraleonte noraleonte

Labels
bug 🐛 Something doesn't work component: tree view TreeView, TreeItem. This is the name of the generic UI component, not the React module! docs Improvements or additions to the documentation
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[tree view][docs] Adjust all demos height to account for the updated design danilo-leal/mui-x
2 participants
@oliviertassinari @noraleonte