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

[docs] New demo of a Tree View and a 3D canvas #13214

Draft
wants to merge 11 commits into
base: master
Choose a base branch
from
Draft

[docs] New demo of a Tree View and a 3D canvas #13214

wants to merge 11 commits into from
+1,226 −0

Conversation

joserodolfofreitas
Copy link
Member

@joserodolfofreitas joserodolfofreitas commented May 22, 2024 •
edited by flaviendelangle
Loading

@joserodolfofreitas joserodolfofreitas added docs Improvements or additions to the documentation component: tree view TreeView, TreeItem. This is the name of the generic UI component, not the React module! labels May 22, 2024
@mui-bot
Copy link

mui-bot commented May 22, 2024 •
edited
Loading

Deploy preview: https://deploy-preview-13214--material-ui-x.netlify.app/

Updated pages:

Generated by 🚫 dangerJS against eb61c60

flaviendelangle
flaviendelangle reviewed May 23, 2024
View reviewed changes
Copy link
Member

@flaviendelangle flaviendelangle left a comment •
edited
Loading

Choose a reason for hiding this comment

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

🥳 🥳 🥳

IMHO the layout would be better optimized with the 3D view below or above the Tree View

image

@github-actions github-actions bot added the PR: out-of-date The pull request has merge conflicts and can't be merged label May 29, 2024
@github-actions GitHub Actions
Copy link

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

@github-actions github-actions bot removed the PR: out-of-date The pull request has merge conflicts and can't be merged label Jun 12, 2024
@flaviendelangle flaviendelangle self-assigned this Jun 12, 2024
@flaviendelangle flaviendelangle changed the title [tree view] Add 3D Canvas demo [docs] New demo of a Tree View and a 3D canvas Jun 12, 2024
@github-actions github-actions bot added the PR: out-of-date The pull request has merge conflicts and can't be merged label Jun 12, 2024
@github-actions GitHub Actions
Copy link

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

@github-actions github-actions bot removed the PR: out-of-date The pull request has merge conflicts and can't be merged label Jun 12, 2024
@flaviendelangle
Copy link
Member

flaviendelangle commented Jun 13, 2024 •
edited
Loading

I'm running into this issue, this fix works I'm trying to apply a clean patch but I guess people will have the same error when pasting the code 😢

@github-actions github-actions bot added the PR: out-of-date The pull request has merge conflicts and can't be merged label Jun 25, 2024
@github-actions 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 PR: out-of-date The pull request has merge conflicts and can't be merged and removed PR: out-of-date The pull request has merge conflicts and can't be merged labels Jul 22, 2024
@github-actions GitHub Actions
Copy link

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

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

@flaviendelangle flaviendelangle flaviendelangle left review comments

At least 1 approving review is required to merge this pull request.

Assignees

@flaviendelangle flaviendelangle

Labels
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 PR: out-of-date The pull request has merge conflicts and can't be merged
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@joserodolfofreitas @mui-bot @flaviendelangle