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

Unstable - RTL issues #4728

Closed
pcenov opened this issue Sep 12, 2024 · 4 comments · Fixed by learningequality/kolibri-design-system#784 or #4735
Closed

Unstable - RTL issues #4728

pcenov opened this issue Sep 12, 2024 · 4 comments · Fixed by learningequality/kolibri-design-system#784 or #4735
Assignees

Comments

@pcenov
Copy link
Member

pcenov commented Sep 12, 2024

Observed behavior

When bulk editing the language of the resources in Arabic the text is not right aligned with the radio buttons. This is also the case at the Content library in the Language filter.

Expected behavior

The strings should be right aligned

Steps to reproduce the issue

See the attached screenshots

Additional information

2024-09-12_15-35-10

2024-09-12_10-37-50

Usage Details

@pcenov
Copy link
Member Author

pcenov commented Sep 12, 2024

@radinamatic

@AlexVelezLl
Copy link
Member

AlexVelezLl commented Sep 12, 2024

I think I saw this some time ago. This is an issue with KCheckbox and KRadioButton applying dir: auto to the label, (e.g. here) and this makes that the alignment takes the dir of the text rather than the dir of the user language. The solution would be to remove that dir prop on the label, but I dont know if this is going to bring any inconsistency somewhere. Do you have any thoughts about this @nucleogenesis @MisRob?

I think for now at least we can add a prop to disable this behaviour.

@akolson akolson added this to the Studio: Bulk Editing Release milestone Sep 12, 2024
@MisRob
Copy link
Member

MisRob commented Sep 13, 2024

Yes @AlexVelezLl, that makes sense to me. Thank you.

@MisRob
Copy link
Member

MisRob commented Sep 13, 2024

Perhaps more general labelDirection or labelDir prop would make sense. We could have it set by default to 'auto' so that it doesn't break any of the existing places, but it would be ready to accept any other dir values such as 'ltr', 'rtl'. This would have benefit of being aligned with the native platform.

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