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

submenu RTL issue #1421

Closed
hosseinfarnia opened this Issue Jan 28, 2018 · 4 comments

Comments

Projects
None yet
3 participants
@hosseinfarnia

hosseinfarnia commented Jan 28, 2018

Hi @Nikeo
multiple Submenus on desktop devices are not aligned to right in RTL languages.
I have provided a screencast for better understanding in the following:
this is the current position in customizr theme which is wrong:
https://screencast-o-matic.com/watch/cFVulyouSP
and this one is the correct position in another theme (Hestia):
https://screencast-o-matic.com/watch/cFVuluouS8
Thank you very much for your attention

@hosseinfarnia

This comment has been minimized.

hosseinfarnia commented Jan 30, 2018

Hi,
I realized that it has been fixed already.
Sorry for opening this issue.
Thank you

@eri-trabiccolo

This comment has been minimized.

Contributor

eri-trabiccolo commented Jan 30, 2018

Hi @hosseinfarnia ,
actually this issue is still valid for the modern style and is not fixed yet. Thanks for reporting this.
This other fixed issue #1414 refers to the background animation on hover for submenu items.

Best.

eri-trabiccolo added a commit to eri-trabiccolo/customizr that referenced this issue Jan 31, 2018

Imp: modern - fix RTL submenu default opening direction
CSS fix for presscustomizr#1421
additionally fix menu button transform in RTL
@eri-trabiccolo

This comment has been minimized.

Contributor

eri-trabiccolo commented Jan 31, 2018

@Nikeo

This comment has been minimized.

Contributor

Nikeo commented Jan 31, 2018

Hi @hosseinfarnia thank you very much for reporting this problem. And for the screecasts, it helped !
@eri-trabiccolo thanks for the PR

@Nikeo Nikeo closed this Feb 2, 2018

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