-
Notifications
You must be signed in to change notification settings - Fork 353
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
RTL - Core followup for RTL changes #9521
Comments
One item that will be needed as part of this: reviewing the components in React and identifying any issues with the final React implementation after pulling in the updated styles from core. |
Just jotting down to keep in mind: DataList and Date Picker examples will have to be checked once/if any update to Popper is implemented to support RTL. DataList has some kebab dropdowns that end up being aligned incorrectly in RTL, DatePicker uses Popover internally which ends up having the arrow positioned incorrectly in RTL. Note: DatePicker may or may not be a Popper issue; possibly related to #9576 DataList example with a dropdown menu in RTL: DatePicker in RTL: |
We'll need to double check Progress once patternfly/patternfly#5923 is merged and pulled into React |
|
The dropdowns in the toolbar demo need to be checked once the dropdown menu is fixed up to switch l/r when there isn't room. https://react-staging.patternfly.org/components/toolbar/react-demos/console-log-viewer-toolbar-demo/ |
All items in this epic have been completed 🎉 |
Goal
The core team is implementing RTL support and some components may require react followup.
This is a placeholder issue until testing is complete.
Check components:
The text was updated successfully, but these errors were encountered: