Skip to content

[core] Allow changing the width of the drawer in dashboard layout#4296

Merged
apedroferreira merged 4 commits intomui:masterfrom
garryxiao:feature/4266-add-sidebar-expanded-width-property
Oct 25, 2024
Merged

[core] Allow changing the width of the drawer in dashboard layout#4296
apedroferreira merged 4 commits intomui:masterfrom
garryxiao:feature/4266-add-sidebar-expanded-width-property

Conversation

@garryxiao
Copy link
Contributor

@garryxiao garryxiao commented Oct 21, 2024

Copy link
Collaborator

@apedroferreira apedroferreira left a comment

Choose a reason for hiding this comment

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

Looks great, thank you!
Can you also please autogenerate the documentation for the new prop?

You should be able to do it by running 2 scripts from the root package after addressing the comments below.

  1. pnpm proptypes
  2. pnpm docs:build:api

And then just add and commit the autogenerated changes!

*/
hideNavigation?: boolean;
/**
* The width in pixel of the sidebar when expanded, default value is 320.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
* The width in pixel of the sidebar when expanded, default value is 320.
* Width of the sidebar when expanded, in pixels.
* @default 320

@apedroferreira apedroferreira self-assigned this Oct 22, 2024
@apedroferreira apedroferreira added component: DashboardLayout type: enhancement It’s an improvement, but we can’t make up our mind whether it's a bug fix or a new feature. labels Oct 22, 2024
Copy link
Collaborator

@apedroferreira apedroferreira left a comment

Choose a reason for hiding this comment

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

Thanks a lot @garryxiao !!

@apedroferreira apedroferreira merged commit 1ef74f0 into mui:master Oct 25, 2024
@apedroferreira apedroferreira changed the title [core] Allow changing the width of the drawer in dashboard layout com… [core] Allow changing the width of the drawer in dashboard layout Oct 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: enhancement It’s an improvement, but we can’t make up our mind whether it's a bug fix or a new feature.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[core] Allow changing the width of the drawer in dashboard layout component

4 participants