-
Notifications
You must be signed in to change notification settings - Fork 402
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
consider removing the RTL flip #48
Comments
Basic question at the end: If people are used to broken designs/behaviour (which may be formally incorrect), should we fix it anyways? I'm unsure if there is a definitive answer in this case at all because it may be very depending on who you ask... The same topic is discussed here https://www.youtube.com/watch?v=OHF2h1_fBCM and I'm quite unsure about this for myself - but would be interested in more input on this. |
great link and thoughts, thanks! |
any progress, to fix it? |
Sidenav don't support RTL |
not clear in this issue, but yeah, it's attached to the Switch component. |
I don't think there is a real issue here, and this can be closed. I have opened the switch component to fix it, but watching the video and interacting with the component in the local environment, you have taken the right approach.
|
Mohammed Ali Agha commented the change is understood but not needed as they've already adjusted
The text was updated successfully, but these errors were encountered: