Skip to content

[material-ui][Badge] Make keys in anchor origin partial#43950

Merged
ZeeshanTamboli merged 10 commits intomui:masterfrom
sai6855:parial-anchor-origin
Oct 3, 2024
Merged

[material-ui][Badge] Make keys in anchor origin partial#43950
ZeeshanTamboli merged 10 commits intomui:masterfrom
sai6855:parial-anchor-origin

Conversation

@sai6855
Copy link
Member

@sai6855 sai6855 commented Oct 1, 2024

closes: #43859

Right now, anchorOrigin prop for Badge expects both keys (horizontal | vertical) to be present when passed, but i don't think there is need to make both keys as required as internally it is possible to handle logic even if one key is passed. This PR is to make keys in anchorOrgin from required to partial

@sai6855 sai6855 added scope: badge Changes related to the badge. package: material-ui 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 1, 2024
@mui-bot
Copy link

mui-bot commented Oct 1, 2024

Netlify deploy preview

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

Bundle size report

Details of bundle changes (Toolpad)
Details of bundle changes

Generated by 🚫 dangerJS against d4b7b5d

@sai6855 sai6855 requested review from ZeeshanTamboli and removed request for ZeeshanTamboli October 1, 2024 07:16
@sai6855 sai6855 marked this pull request as draft October 1, 2024 07:26
@sai6855 sai6855 marked this pull request as ready for review October 1, 2024 11:09
@sai6855 sai6855 requested a review from ZeeshanTamboli October 1, 2024 11:10
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.

Looks good.

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

Labels

scope: badge Changes related to the badge. 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.

[docs] Badge anchorOrigin should mention the required keys

3 participants