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

fix: Fix computing of RTL based Language for Card Carousel Arrows - MEED-7912 - Meeds-io/meeds#2652 #4252

Merged
merged 1 commit into from
Dec 9, 2024

Conversation

boubaker
Copy link
Member

@boubaker boubaker commented Dec 9, 2024

Prior to his change, some changes were made to make the Card carousel Arrows works on RTL based Locales which had made some regressions on LTR based languages. This change will revert recent changes and rework the RTL computing algorithms to consider RTL DOM offset computing.

Resolves Meeds-io/meeds#2652

@boubaker boubaker requested a review from AzmiTouil December 9, 2024 09:14
@boubaker boubaker enabled auto-merge (squash) December 9, 2024 09:14
@github-actions github-actions bot added the partialCIBuild Perform Partial CI Build label Dec 9, 2024
…ows - MEED-7912 - Meeds-io/meeds#2652

Prio tot his change, some changes were made to make the Card caroussel Arrows works on RTL based Locales which had made some regressions on LTR based languages. This change will revert recent changes and rework the RTL computing algorithms to consider RTL Javascript offset computing.
@boubaker boubaker merged commit 59fc2a0 into develop Dec 9, 2024
3 checks passed
@boubaker boubaker deleted the MEED-7861 branch December 9, 2024 10:35
Copy link

sonarcloud bot commented Dec 9, 2024

rdenarie pushed a commit that referenced this pull request Dec 9, 2024
…EED-7912 - Meeds-io/meeds#2652 (#4252)

Prior to his change, some changes were made to make the Card carousel
Arrows works on RTL based Locales which had made some regressions on LTR
based languages. This change will revert recent changes and rework the
RTL computing algorithms to consider RTL DOM offset computing.

Resolves Meeds-io/meeds#2652
rdenarie pushed a commit that referenced this pull request Dec 9, 2024
…EED-7912 - Meeds-io/meeds#2652 (#4252)

Prior to his change, some changes were made to make the Card carousel
Arrows works on RTL based Locales which had made some regressions on LTR
based languages. This change will revert recent changes and rework the
RTL computing algorithms to consider RTL DOM offset computing.

Resolves Meeds-io/meeds#2652
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
partialCIBuild Perform Partial CI Build
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Card Carousel doesn't work anymore on Badges
2 participants