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

[css-images] RTL issue in radial-gradient() and conic-gradient() #6270

Open
ramiy opened this issue May 5, 2021 · 3 comments
Open

[css-images] RTL issue in radial-gradient() and conic-gradient() #6270

ramiy opened this issue May 5, 2021 · 3 comments
Labels
css-images-4 i18n-alreq Arabic language enablement i18n-hlreq Hebrew language enablement i18n-tracker Group bringing to attention of Internationalization, or tracked by i18n but not needing response.

Comments

@ramiy
Copy link

ramiy commented May 5, 2021

Both Radial Gradients and Conic Gradients are not RTL compatible. When using RTL languages, we need the reverse everything.

Currently both radial-gradient() and conic-gradient() support only one direction, clockwise. There is not support for counter-clockwise.

There should be a way to set counter-clockwise direction.

Possible solution

In redial-gradient we have keywords like closest-side, closest-corner, farthest-side and farthest-corner.

There should be two "direction" keywords clockwise and counter-clockwise. Why two? do override previously defined settings.

@ramiy
Copy link
Author

ramiy commented May 5, 2021

@LeaVerou please share your thoughts on this issue.

@LeaVerou
Copy link
Member

LeaVerou commented May 6, 2021

Yeah, we definitely need logical keywords in addition to the physical ones already there.

I'm confused about the clockwise and counter-clockwise proposal. Is that a separate proposal or does it somehow help with bidi?

@ramiy
Copy link
Author

ramiy commented May 6, 2021

To clarify my solution, I was talking about something like this:

background: conic-gradient(at 50% 50% clockwise, #f06, gold);
background: conic-gradient(at 50% 50% counter-clockwise, #f06, gold);

background: conic-gradient(from 45deg clockwise, white, black, white);
background: conic-gradient(from 45deg counter-clockwise, white, black, white);

You can use other keywords or propose other solutions.

As long as I'll able to reverse everything, I don't care about the "how".

@svgeesus svgeesus added the i18n-tracker Group bringing to attention of Internationalization, or tracked by i18n but not needing response. label Oct 16, 2021
@xfq xfq added i18n-alreq Arabic language enablement i18n-hlreq Hebrew language enablement labels Oct 18, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
css-images-4 i18n-alreq Arabic language enablement i18n-hlreq Hebrew language enablement i18n-tracker Group bringing to attention of Internationalization, or tracked by i18n but not needing response.
Projects
None yet
Development

No branches or pull requests

5 participants