-
-
Notifications
You must be signed in to change notification settings - Fork 166
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
RTL Layout Support #4167
Comments
if it all, it should be a debug-only-hidden setting, not even experimental. for the user, there is no reason to have this option - languages are RTL or LRT, not both. android/iOS/Windows/Mac/whatever usually do not have this option in UI accessible by the user. so, i would not go for that at all - for testing, devs can easily just switch to an RTL language - we devs anyway know the position of buttons without reading :)
makes sense esp. for the "back" button, as you already noted. for all other icons, i would defer these discussions when all other parts are fine. is there just a CSS trick for mirroring? that would make the decision easier, however (cmp. https://m2.material.io/design/usability/bidirectionality.html#mirroring-layout )
i would expect that to make sense as well, however, this is again a minor issue, compared to the overall fix of showing the layout correctly mirrored NB: i added the "bug" label as we already have at least some RTL languages that are displayed wrong |
The thing is, that there is no conseneus on the Persian community that the layout should be RTL. Actually some folks find it "weird" after a lifetime of working with LTR layout in the softwares. Needless to say, Every single ordinary Persian user uses the computers with English UI language, and LTR. Just advanced users switch to Persian UI language for their computers. Users like me. So keep in mind that:
So overall: adding RTL support is definitely a huge plus for Iranians but not as big impact as it has for Hebrew and Arabic speakers. Here in my comments, by Persians I mean Iranian Persians. Tajiks and Afghans respect their language and culture much more than we do. |
thanks a lot for the insights and details!
is there a comparable user-changable option in Android/iOS/Windows/Mac or other larger apps? surely, one can think of whatever cases, however, it also makes sense to keep things simple. for the beginning, i would not go beyond what is usual and offered by other systems and app, that could go to a feature proposal on android/iOS, Farsi is RTL since forever, and iirc, no one complained; desktop adding more stuff here may raise expectations also there. so also consistency should be taken into account
maybe add a second locale than, that transcribes Farsi in Latin? is there a language code for that? cmp deltachat/deltachat-android#3213 where we have troubles with that ... :) |
The standard form for the app is to be mirrored. But some people within the Persian community don't like this "standard" on PC. On Android people have been using Persian UI with mirrored UI since the first day so there is no problem there. I agree it's weird that some dislike this. But it is how it is. There are other weird things within the community:
I'm pinging @ahangarha @mehr32 to get them involved. |
A general point: But to me, the issue is not adding RTL support. All apps should have bidirectional text support. This means, regardless of the overall layout, any element with user generated content should show the content (and also apply padding, margin, border) based on its own content. And this is very much doable with all major technologies used today. Regarding using Latin alphabet for Persian, I can say do not waste your time on it. I don't have even a single example in mind where an app supports Persian with Latin characters. This is just a waste of time. Either add support for Persian with its official character sets (unicode standard) or just focus on other features of the app. To @farooqkz: |
Areas:
margin-right
,padding-left
,float: right
and so on and convert toinline-start
/inline-end
Useful resources:
dir=<ltr|rtl>
attributeQuestions:
Misc:
_locales/_languages.json
?CC @farooqkz
The text was updated successfully, but these errors were encountered: