Skip to content

feat: modernize rtl#4445

Open
pdanpdan wants to merge 4 commits intosaadeghi:masterfrom
pdanpdan:feat/modernize-rtl
Open

feat: modernize rtl#4445
pdanpdan wants to merge 4 commits intosaadeghi:masterfrom
pdanpdan:feat/modernize-rtl

Conversation

@pdanpdan
Copy link
Contributor

  • use :dir(rtl) instead of [dir="rtl"]
  • use logical directions in a few places

@saadeghi
Copy link
Owner

This can be a potential breaking change for some people, if they are nesting different directions. I need to do more testing before merging this

@pdanpdan
Copy link
Contributor Author

Yes, I agree - if they nested different directions it is a breaking fix :)
Maybe in the next minor release, because I would qualify this as a fix in order to get the expected behavior (and not as a changed behavior).

@saadeghi saadeghi added the 5.6 label Feb 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants