Skip to content

v5.8.8#33468

Closed
siriwatknp wants to merge 2 commits intomui:masterfrom
siriwatknp:release/5.8.8
Closed

v5.8.8#33468
siriwatknp wants to merge 2 commits intomui:masterfrom
siriwatknp:release/5.8.8

Conversation

@siriwatknp
Copy link
Member

@siriwatknp siriwatknp added the release We are shipping :D. label Jul 11, 2022
@siriwatknp siriwatknp requested a review from a team July 11, 2022 11:29
@mui-bot
Copy link

mui-bot commented Jul 11, 2022

No bundle size changes

Generated by 🚫 dangerJS against 3c6d143

### `@mui/base@5.0.0-alpha.89`

- &#8203;<!-- 29 -->[Base] Add MUI base example (#33154) @siriwatknp
- &#8203;<!-- 28 —>[Base] Change the order of class names merged in useSlotProps (#33383) @michaldudak
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
- &#8203;<!-- 28 >[Base] Change the order of class names merged in useSlotProps (#33383) @michaldudak
- &#8203;<!-- 28 -->[Base] Change the order of class names merged in useSlotProps (#33383) @michaldudak

or just remove the prefixes altogether

Copy link
Member Author

Choose a reason for hiding this comment

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

I'll remove them all.

- &#8203;<!-- 19 -->[docs] Add "refine" demo to showcase (#33240) @omeraplak
- &#8203;<!-- 18 -->[docs] Add webpack alias for legacy utils package (#33376) @jgbae
- &#8203;<!-- 16 -->[docs] Improve external link icons synonyms (#33257) @davidgarciab
- &#8203;<!-- 31 —>[docs] Correcting small grammatical error (#33393) @robyyo
Copy link
Member

Choose a reason for hiding this comment

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

It seems like whatever editor you're using thinks it's smarter ;)

Suggested change
- &#8203;<!-- 31 >[docs] Correcting small grammatical error (#33393) @robyyo
- &#8203;<!-- 31 -->[docs] Correcting small grammatical error (#33393) @robyyo

@siriwatknp
Copy link
Member Author

closed this in favor of #33483

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

Labels

release We are shipping :D.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants