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

[pickers] Fix TimeClock meridiem button selected styles #16681

Merged
merged 2 commits into from
Feb 21, 2025

Conversation

LukasTy
Copy link
Member

@LukasTy LukasTy commented Feb 21, 2025

It's broken only on v8/master.

Before After
Screenshot 2025-02-21 at 12 31 39 Screenshot 2025-02-21 at 12 48 03

@LukasTy LukasTy added bug 🐛 Something doesn't work component: pickers This is the name of the generic UI component, not the React module! labels Feb 21, 2025
@LukasTy LukasTy self-assigned this Feb 21, 2025
@mui-bot
Copy link

mui-bot commented Feb 21, 2025

Deploy preview: https://deploy-preview-16681--material-ui-x.netlify.app/

Generated by 🚫 dangerJS against 69de7d3

@@ -84,7 +84,7 @@ const ClockPointerThumb = styled('div', {
boxSizing: 'content-box',
variants: [
{
props: { isBetweenTwoClockValues: false },
props: { isClockPointerBetweenTwoValues: false },
Copy link
Member Author

Choose a reason for hiding this comment

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

This one fixes the unwanted white dot on the selected hour thumb. 🙈
@siriwatknp Am I missing something or there is no way to infer variants props keys types? 🤔
These regressions are annoying and really hard to spot. 🙈

@LukasTy LukasTy merged commit 016fe09 into mui:master Feb 21, 2025
18 checks passed
@LukasTy LukasTy deleted the fix-time-clock-meridiem-button-styles branch February 21, 2025 11:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🐛 Something doesn't work component: pickers This is the name of the generic UI component, not the React module!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants