-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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] MultiSectionDigitalClock
incorrect RTL behavior
#11525
Comments
Hello @Eliav2. Thank you for creating this issue! 🙏 |
MultiSectionDigitalClock
RTL behavior
MultiSectionDigitalClock
RTL behaviorMultiSectionDigitalClock
incorrect RTL behavior
Yes, exactly. I would even suggest considering adding an option to pass props to any inner element to allow full customization including overriding styles |
Thank you for clarifying. 👍
There is the |
Our ux decided to proceed with different ui but this component is awesome and a true RTL support would benefit to others for sure(i have not trying override styles with slots) |
@Eliav2, I'm curious about your UX team's decision. What are you using now instead of the MUI X time picker? |
Because we focus on relative time(eg hours,minutes ,seconds from now) we just used 3 mui Select components to represent hours,minutes, seconds. @joserodolfofreitas |
@Eliav2 would you be able to clarify the expected tabulation order for the cc @MBilalShafi if you can confirm this. 🙏 |
Hours should be on the left. Minutes right to them. Seconds right to them. Time are internetional. The resolution is as simple as adding sx={direction:ltr} on the right div. Because direction ordering should never change on nunbers. They are always the same. |
Hey @Eliav2, thanks for the suggestion! |
@Eliav2: How did we do? Your experience with our support team matters to us. If you have a moment, please share your thoughts in this short Support Satisfaction survey. |
Originally posted by @Eliav2 in #5561 (comment)
heres a complete sandbox showing the issue.
we can see that the hours and minutes are on the opposite side of what is expected. this sandbox implements MUI RTL theme as the docs describe.
Search keywords:
The text was updated successfully, but these errors were encountered: