Skip to content

[docs][Menu] Update paper slot JSDoc default from Paper to PopoverPaper #45722

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

Merged
merged 4 commits into from
Apr 10, 2025

Conversation

andreachiera
Copy link
Contributor

The PopoverPaper is the real instance used inside the component. If you try to just apply a new Paper instance to the slot, the UI is broken in terms of positioning.

The PopoverPaper is the real instance. In fact, if you try to just apply a new Paper instance to the slot, the UI is broken in terms of paper positioning.

Signed-off-by: Andrea Chiera <[email protected]>
@mui-bot
Copy link

mui-bot commented Mar 28, 2025

Netlify deploy preview

https://deploy-preview-45722--material-ui.netlify.app/

Bundle size report

No bundle size changes (Toolpad)
No bundle size changes

Generated by 🚫 dangerJS against 9645ec6

@andreachiera
Copy link
Contributor Author

Since there are issues in compiling the documentation, I could add this information inside the Description column?

@andreachiera andreachiera changed the title fix: menu documentation [material-ui][menu] documentation Mar 28, 2025
Copy link
Member

@mnajdova mnajdova left a comment

Choose a reason for hiding this comment

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

This json files are automatically generated out of the types, we will need to change this: https://github.com/mui/material-ui/blob/master/packages/mui-material/src/Menu/Menu.d.ts#L32 instead and run

pnpm proptypes
pnpm docs:api

@mnajdova mnajdova added docs Improvements or additions to the documentation component: menu This is the name of the generic UI component, not the React module! labels Mar 28, 2025
@ZeeshanTamboli ZeeshanTamboli changed the title [material-ui][menu] documentation [docs][Menu] Update default Menu Paper documentation Mar 30, 2025
@ZeeshanTamboli ZeeshanTamboli changed the title [docs][Menu] Update default Menu Paper documentation [docs][Menu] Update JSDoc for default Menu Paper Mar 30, 2025
@ZeeshanTamboli ZeeshanTamboli added v6.x needs cherry-pick The PR should be cherry-picked to master after merge labels Apr 10, 2025
@ZeeshanTamboli ZeeshanTamboli changed the title [docs][Menu] Update JSDoc for default Menu Paper [docs][Menu] Update paper slot JSDoc default from Paper to PopoverPaper Apr 10, 2025
Copy link
Member

@ZeeshanTamboli ZeeshanTamboli left a comment

Choose a reason for hiding this comment

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

@andreachiera Thanks for the pull request. I made the expected changes.

@ZeeshanTamboli ZeeshanTamboli merged commit 42cc8ea into mui:master Apr 10, 2025
22 checks passed
Copy link

Cherry-pick PRs will be created targeting branches: v6.x

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: menu This is the name of the generic UI component, not the React module! docs Improvements or additions to the documentation needs cherry-pick The PR should be cherry-picked to master after merge v6.x
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants