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(rtl): change ltr text-align to rtl-compatible form #761

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

maxpowa
Copy link

@maxpowa maxpowa commented Nov 14, 2023

Changes proposed in this pull request:

  • Change a text-align: left to text-align: start to increase RTL compatibility.

Acceptance Criteria

General

  • Tested on desktop (see supported browsers)
  • Tested on mobile (for responsive or mobile-specific features)
  • Confirm Accessibility
  • Confirm RTL (how did this work before this change???)

Copy link

Thanks for the contribution! Before we can merge this, we need @maxpowa to sign the Salesforce Inc. Contributor License Agreement.

@maxpowa
Copy link
Author

maxpowa commented Nov 14, 2023

Not sure whats up with the @salesforce-cla bot, I've signed the CLA and it hasn't updated...

Edit: Oopsie, used the wrong author email 😭

@Dottenpixel
Copy link
Contributor

Hi @maxpowa Thank you for your contribution! With these changes affect the CSS, please allow us some time to internally run visual regression testing. I've created an internal ticket to shepherd this through our workflow.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants