Skip to content
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

switch to start/end instead of left/right to support RTL layouts #498

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

ibrasho
Copy link

@ibrasho ibrasho commented Oct 21, 2024

This PR makes some changes to the tailwind classes used to support RTL layouts.

I'm applying similar changes to my local version of the pro files.

Copy link

@cs4alhaider cs4alhaider left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @ibrasho for this PR!

Just to give a context as this will improve the adaptability for different languages as using left and right is static—it assumes all languages are left-to-right (LTR), like English. But in right-to-left (RTL) languages like Arabic or Hebrew, layouts are often mirrored. start and end dynamically adjust based on the text direction.
start aligns to the left in LTR languages and to the right in RTL languages. Similarly, end aligns to the right in LTR and to the left in RTL.
It also for internationalization (i18n) as this change is crucial for supporting applications used by people from different cultures and languages. By using start and end, the layout automatically adjusts to the user's language preference, improving usability for RTL users.

Long time no seee @ibrasho 🙋🏻‍♂️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants