-
Notifications
You must be signed in to change notification settings - Fork 151
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
feat(rtl-arrow): Fix arrows used for CTA's when in RTL #11736
Conversation
Deploy preview created for package Built with commit: 83b35f4c0b5fe89eb17262d9263f67255e07d1ab |
Deploy preview created for package Built with commit: 83b35f4c0b5fe89eb17262d9263f67255e07d1ab |
Deploy preview created for package Built with commit: fdad5869a9b07edac26cdeb8e423c0523519b2c3 |
Deploy preview created for package Built with commit: 83b35f4c0b5fe89eb17262d9263f67255e07d1ab |
Deploy preview created for package Built with commit: 83b35f4c0b5fe89eb17262d9263f67255e07d1ab |
Deploy preview created for package Built with commit: 83b35f4c0b5fe89eb17262d9263f67255e07d1ab |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Everything about this look good from a code perspective. Nice work as usual, @m4olivei!
I did find some Stories where arrows still aren't pointing the correct way in RTL. I imagine these are probably content issues in the Stories rather than code issues, but calling them out for a double check:
Deploy preview created for package Built with commit: 83b35f4c0b5fe89eb17262d9263f67255e07d1ab |
Thanks @jkaeser ! I've cleaned up those additional stories that you pointed out. Ready for another look. |
Deploy preview created for package Built with commit: 83b35f4c0b5fe89eb17262d9263f67255e07d1ab |
23f196f
into
carbon-design-system:main
Hey there! This issue/pull request was referenced in recently released v2.9.0. |
Related Ticket(s)
Closes #11725
Jira Ticket
Description
Checkes
document.dir
and uses the appropriate arrow icon for the text direction (right in LTR and left in RTL). For many stories involving CTA's or CTA derivatives, avoid using the slotted icon when all we're going for iscta-type="local"
, in other words the arrow icon.Testing instructions
Changelog
Changed