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

Bump MUI Core #4601

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Bump MUI Core #4601

wants to merge 4 commits into from

Conversation

renovate[bot]
Copy link
Contributor

@renovate renovate bot commented Jan 14, 2025

This PR contains the following updates:

Package Change Age Adoption Passing Confidence
@mui/docs (source) 6.3.1 -> 6.4.0 age adoption passing confidence
@mui/icons-material (source) 6.3.1 -> 6.4.0 age adoption passing confidence
@mui/internal-markdown (source) 1.0.24 -> 1.0.25 age adoption passing confidence
@mui/internal-scripts (source) 1.0.32 -> 1.0.33 age adoption passing confidence
@mui/lab (source) 6.0.0-beta.22 -> 6.0.0-beta.23 age adoption passing confidence
@mui/material (source) 6.3.1 -> 6.4.0 age adoption passing confidence
@mui/styles (source) 6.3.1 -> 6.4.0 age adoption passing confidence
@mui/system (source) 6.3.1 -> 6.4.0 age adoption passing confidence
@mui/utils (source) 6.3.1 -> 6.4.0 age adoption passing confidence

Release Notes

mui/material-ui (@​mui/docs)

v6.4.0

Compare Source

Jan 13, 2025

A big thanks to the 16 contributors who made this release possible. Here are some highlights ✨:

@mui/[email protected]
Docs
Core

All contributors of this release in alphabetical order: @​aarongarciah, @​DiegoAndai, @​good-jinu, @​headironc, @​iaziz11, @​IceOfSummer, @​Janpot, @​LukasTy, @​mnajdova, @​oliviertassinari, @​sai6855, @​samuelsycamore, @​siddhantantil39, @​siriwatknp, @​yashdev16, @​ZeeshanTamboli


Configuration

📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.


  • If you want to rebase/retry this PR, check this box

This PR was generated by Mend Renovate. View the repository job log.

@renovate renovate bot added the dependencies Update of dependencies label Jan 14, 2025
@mui-bot
Copy link

mui-bot commented Jan 14, 2025

Netlify deploy preview

https://deploy-preview-4601--mui-toolpad-docs.netlify.app/

Generated by 🚫 dangerJS against e67d340

@github-actions github-actions bot added the PR: out-of-date The pull request has merge conflicts and can't be merged label Jan 15, 2025
@renovate renovate bot force-pushed the renovate/mui-core branch from 762d396 to 4cdc68c Compare January 15, 2025 05:57
@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 Jan 15, 2025
@renovate renovate bot force-pushed the renovate/mui-core branch from 4cdc68c to a8edde9 Compare January 15, 2025 06:18
@github-actions github-actions bot removed the PR: out-of-date The pull request has merge conflicts and can't be merged label Jan 15, 2025
Copy link
Contributor Author

renovate bot commented Jan 16, 2025

Edited/Blocked Notification

Renovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR.

You can manually request rebase by checking the rebase/retry box above.

⚠️ Warning: custom changes will be lost.

@@ -111,7 +111,7 @@ function PagesExplorerTreeItem(props: StyledTreeItemProps) {
})}
aria-label="Open page explorer menu"
size="small"
{...buttonProps}
{...(buttonProps as IconButtonProps)}
Copy link
Member

Choose a reason for hiding this comment

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

Avoid casting, instead fix the type of renderButton in NodeMenu

@@ -28,7 +35,7 @@ function AppOptions({ dom }: AppOptionsProps) {

return (
<React.Fragment>
<IconButton {...buttonProps} aria-label="Application menu">
<IconButton {...(buttonProps as IconButtonProps)} aria-label="Application menu">
Copy link
Member

Choose a reason for hiding this comment

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

We should probably fix the types of useMenu instead so that they work for both Button and IconButton.

Copy link
Member

@bharatkashyap bharatkashyap Jan 16, 2025

Choose a reason for hiding this comment

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

If I understand correctly, the issue has arisen because of the mismatch in the type of the loading prop between IconButton and Button and everything should work as expected when this issue is resolved without us needing to do anything

Copy link
Member

Choose a reason for hiding this comment

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

Looks like it. Is there a core issue? We can hold off on upgrading until the issue is resolved

Copy link
Member

Choose a reason for hiding this comment

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

Can't seem to find an issue for this. @siriwatknp is the difference in the types of IconButton and Button wrt loading intentional? (See my comment above in the thread for the links)

@github-actions github-actions bot added the PR: out-of-date The pull request has merge conflicts and can't be merged label Jan 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Update of dependencies PR: out-of-date The pull request has merge conflicts and can't be merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants