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

Implement metadata panel new design #687

Merged
merged 42 commits into from
Dec 16, 2024
Merged

Implement metadata panel new design #687

merged 42 commits into from
Dec 16, 2024

Conversation

guergana
Copy link
Collaborator

@guergana guergana commented Dec 11, 2024

This PR implements the redesign of the tabs navigation of the metadata panel. By changing the MenuTree component to MuiTabs the complexity of the code is reduced. Some very small details are missing but still can be reviewed.

TODO:
Very small details: change color of trash can icon (delete) in ListItems (Contributors, Fields, Licenses, Foreign Keys) and change border color for newly added items. Will create another PR for this.

@guergana guergana marked this pull request as draft December 11, 2024 21:38
Copy link

cloudflare-workers-and-pages bot commented Dec 14, 2024

Deploying opendataeditor with  Cloudflare Pages  Cloudflare Pages

Latest commit: ee536ab
Status: ✅  Deploy successful!
Preview URL: https://3e897478.opendataeditor.pages.dev
Branch Preview URL: https://664-metadata-pane.opendataeditor.pages.dev

View logs

@guergana guergana marked this pull request as ready for review December 14, 2024 20:26
Copy link
Collaborator

@romicolman romicolman left a comment

Choose a reason for hiding this comment

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

Approved :)

Copy link
Collaborator

@roll roll left a comment

Choose a reason for hiding this comment

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

Looks good in-general. Not sure if "Default" section is the right name though

@guergana
Copy link
Collaborator Author

Looks good in-general. Not sure if "Default" section is the right name though

@roll in the design it was decided this is the name. I guess @romicolman has checked this already.

@guergana guergana changed the base branch from main to 271/getting-started December 16, 2024 16:41
@guergana guergana changed the base branch from 271/getting-started to main December 16, 2024 16:42
Though it's still a bit flimsy
@guergana guergana merged commit 0418ab5 into main Dec 16, 2024
9 checks passed
@guergana guergana deleted the 664-metadata-pane branch December 16, 2024 17:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

The New Metadata panel design
3 participants