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

RTL - Core followup for RTL changes #9521

Closed
83 tasks done
Tracked by #65
andrew-ronaldson opened this issue Aug 22, 2023 · 6 comments
Closed
83 tasks done
Tracked by #65

RTL - Core followup for RTL changes #9521

andrew-ronaldson opened this issue Aug 22, 2023 · 6 comments

Comments

@andrew-ronaldson
Copy link
Collaborator

andrew-ronaldson commented Aug 22, 2023

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:

@wise-king-sullyman
Copy link
Contributor

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.

@thatblindgeye thatblindgeye moved this from Not started to In Progress in PatternFly Issues Sep 11, 2023
@thatblindgeye
Copy link
Contributor

thatblindgeye commented Sep 11, 2023

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:

image

DatePicker in RTL:

image

@tlabaj tlabaj added the Epic label Sep 11, 2023
@thatblindgeye
Copy link
Contributor

We'll need to double check Progress once patternfly/patternfly#5923 is merged and pulled into React

@mcoker
Copy link
Contributor

mcoker commented Sep 13, 2023

@kmcfaul currently, there is a CSS var --pf-v5-global--inverse--multiplier with a value set to 1 or -1 depending on whether you're in an LTR or RTL block via the dir attribute value or using .pf-v5-m-dir-[ltr/rtl]. The CSS property direction should always return "ltr" if you're in an LTR block or "RTL" if in an RTL block. You can get the val using this technique https://developer.mozilla.org/en-US/docs/Web/CSS/Using_CSS_custom_properties#values_in_javascript

getComputedStyle(element).getPropertyValue("direction")

@srambach
Copy link
Member

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/

@wise-king-sullyman
Copy link
Contributor

All items in this epic have been completed 🎉

@github-project-automation github-project-automation bot moved this from In Progress - on schedule to In Progress - precarious in PF Roadmap Sep 21, 2023
@github-project-automation github-project-automation bot moved this from In Progress to Done in PatternFly Issues Sep 21, 2023
@srambach srambach moved this from In Progress - precarious to Done in PF Roadmap Oct 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Archived in project
Development

No branches or pull requests

7 participants