-
-
Notifications
You must be signed in to change notification settings - Fork 666
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
The ROW
direction in RTL Android Phones is a mess!
#2692
Comments
Thanks for the report. I don't doubt there are issues with RTL handling; sadly, none of the currently active core team members are fluent in an RTL language. It would be especially helpful if you're able to provide examples of layouts that you know to be problematic, with code samples. There are parts of RTL handling that are implemented as cross-platform layout code, and some that leverage platform RTL handling. Having specific examples of problematic layouts provides specific issues that can be resolved, rather than a nebulous "it's a mess" descriptor which, while it may be true, doesn't provide a very good mechanism for us to determine when enough fixes have been applied that it is no longer "a mess". |
I see the same on API level 34. It can be worked around in your app by editing build/.../android/gradle/app/src/main/AndroidManifest.xml to change We probably shouldn't make this change in the Briefcase template, because it would be difficult to remove in the future given that there's not a direct relationship between Toga and Briefcase versions. Instead, we should try and fix Toga so it's at least usable in RTL locales until we support them fully. |
Toga.Box()
on the ROW
direction in RTL Android Phones is a mess!ROW
direction in RTL Android Phones is a mess!
Thanks for the suggestion. |
Sorry, I got that the wrong way round. Try setting it to "false". That fixed your example code for me, on a device set to Arabic. But expect it will still have some issues in the presence of actual Arabic text, so please try it and let us know what you think. |
Thank you very much, it is way better now. |
No, I changed my mind about that – see this comment. |
Describe the bug
Toga.Box()
on theROW
direction in RTL Android Phones is a mess, while it is working just fine in LTR.Steps to reproduce
git@github.com:vzool/zakat-tracker.git
Expected behavior
The UI should behave the same on all RTL or LTR locale phones.
Screenshots
On the left is the issue on RTL Android phone languages like Arabic, on the right is the same UI on LTR Android phone languages like English:
is
Environment
Logs
Additional context
No response
The text was updated successfully, but these errors were encountered: