Hello..
There is a problem with the CSS of the main menu with the arabic language ( any RTL language either i guess =

CommentFileSizeAuthor
----.png49.13 KBaminebourkadi

Comments

danny englander’s picture

Component: User interface » Code
Assigned: aminebourkadi » danny englander
Category: bug » support
Status: Active » Postponed (maintainer needs more info)

Hi, I haven't actually set anything up for RTL and it was not on the roadmap. That being said, is there any way you can send me a private message with the URL if you have it online and I can take a quick look? Otherwise it's hard to tell from the screen capture what's going on.

aminebourkadi’s picture

OK Thank you..
Here is the URL :yakada.org , you can see that in the main menu the background color didn'r fit .
Thank you very much

danny englander’s picture

As far as I can tell, you do not have jQuery Update installed and configured properly. Please refer to the project page or documentation on how to do this.

aminebourkadi’s picture

Even after installing the jquery_update module and set it to 1.7 version , nothing happens

danny englander’s picture

Category: support » bug
Status: Postponed (maintainer needs more info) » Needs work

Ok, I see the problem now. There's a Drupal core system RTL css file that's messing with the RTL menu. The lines of code are:

li.expanded, li.collapsed, li.leaf {
padding: 0.2em 0 0 0.5em;
}

... coming from:

system.menus-rtl.css

I can push up a fix for this or in the mean time you could set the padding to 0 for that code in a local.css file (2 ways to do this, see the docs) for the theme. So it would be:

li.expanded, li.collapsed, li.leaf {
padding: 0;
}

At any rate, I will push a fix up in my next go around which would probably be soon. Thanks for reporting this. It would be nice to have full RTL support in the theme but I'm not sure when I might have time to do that but I can do these little fixes in the mean time.

aminebourkadi’s picture

Welcome,thank you for this perfect theme, it would be perfect without this small bug

danny englander’s picture

Status: Needs work » Needs review

The menu issue should be fixed now with https://drupal.org/node/2053337. Please let me know. Thanks.

danny englander’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.