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

Range control breaks functionality at the styling level. #35208

Closed
HILAYTRIVEDI opened this issue Sep 29, 2021 · 3 comments
Closed

Range control breaks functionality at the styling level. #35208

HILAYTRIVEDI opened this issue Sep 29, 2021 · 3 comments
Labels
[Status] Duplicate Used to indicate that a current issue matches an existing one and can be closed

Comments

@HILAYTRIVEDI
Copy link
Contributor

Description

Description
While working on #34855, I noticed in RTL languages the range control stylings are messed up. They are still reflecting the LTR stylings instead of RTL.

Step-by-step reproduction instructions

Step-by-step reproduction instructions
With WordPress set to an RTL language, interact with a range control and observe the stylings.

Screenshots, screen recording, code snippet

https://www.awesomescreenshot.com/video/5421344?key=131c60abb9cd1554b0adbf3d9205fd7b

Environment info

Gutenberg: 11.5.0-rc.1
WordPress: 5.8.1

Please confirm that you have searched existing issues in the repo.

Yes

Please confirm that you have tested with all plugins deactivated except Gutenberg.

Yes

@carolinan
Copy link
Contributor

Hi,
does this only happen in Twenty Twenty-One or in other themes too?

@HILAYTRIVEDI
Copy link
Contributor Author

Hi, does this only happen in Twenty Twenty-One or in other themes too?

Hello @carolinan, yes it is happening in all the themes. So I guess it is problem with the component itself.

@HILAYTRIVEDI HILAYTRIVEDI changed the title Range control in the Twenty Twenty-one theme breaks functionality at the styling level. Range control breaks functionality at the styling level. Sep 29, 2021
@stokesman stokesman added the [Status] Duplicate Used to indicate that a current issue matches an existing one and can be closed label Oct 10, 2021
@stokesman
Copy link
Contributor

Hi @HILAYTRIVEDI. It looks like this is a duplicate of #29519. Which even has an open PR to fix it but I think it's gone stale because with actual RTL language use there is no issue. It's more of an issue with testing RTL in a less complete way.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Status] Duplicate Used to indicate that a current issue matches an existing one and can be closed
Projects
None yet
Development

No branches or pull requests

3 participants