-
Notifications
You must be signed in to change notification settings - Fork 3.3k
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
Properly support RTL languages on web #916
Comments
Great point, supporting RTL languages is crucial for providing a seamless user experience to our global audience. I have experience in implementing RTL support on web applications and would be happy to help out with this issue. I suggest using a CSS framework like Bootstrap or Foundation that already have built-in support for RTL languages, this would make the process more efficient and save time. Additionally, we should also test thoroughly to ensure all elements are properly aligned and all components are functioning correctly in RTL mode. I suggest starting with Arabic as a test language and then expanding the list once we are confident that the UI is properly switching into RTL mode. Let me know if there's anything else I can do to help with this issue. |
I think someone might have implemented this |
still need to fix padding, margins and such... |
Fixes #3199 Refs #916 Still a bit buggy, because the editor itself is not designed to do rtl, but works for now ![image](https://github.com/LAION-AI/Open-Assistant/assets/24505302/e772e3f7-c36e-453b-b1df-05a05a5f8915) ![image](https://github.com/LAION-AI/Open-Assistant/assets/24505302/9e4c515b-8d46-424c-8061-945d5bc3dc2b)
Before we add a RTL language as a supported language, we need to ensure the UI properly switches into RTL mode and all components behavior and render correctly.
To test we can use
ar
(Arabic). Once done, we can add RTL languages to the list.The text was updated successfully, but these errors were encountered: