WordPress.org

Make WordPress Core

Opened 9 years ago

Closed 9 years ago

#18864 closed defect (bug) (worksforme)

Menus doesn't RTL in twently eleven

Reported by: walid3 Owned by:
Milestone: Priority: normal
Severity: normal Version: 3.3
Component: RTL Keywords:
Focuses: Cc:

Description

when using Arabic language as main blog language with twenty eleven theme, the menus bar doesn't RTL, it show exactly as it shows in English theme.

Attachments (2)

rtl-menus.gif (5.4 KB) - added by walid3 9 years ago.
RTL menus screenshot
18864.twenty-eleven.png (80.3 KB) - added by SergeyBiryukov 9 years ago.

Download all attachments as: .zip

Change History (10)

#1 @SergeyBiryukov
9 years ago

RTL layout can be selected in theme options.

There were also some discussions on this in #17603 and #17882.

@walid3
9 years ago

RTL menus screenshot

#2 @walid3
9 years ago

@SergeyBirykuf Yes, I have chosen the RTL layout already, everything is working well except the menus bar as showing in the screenshot I attached now, the menu start from left to right, I read through the two topics but nothing about menus there.

#3 @SergeyBiryukov
9 years ago

  • Component changed from General to RTL

Ah, I was thinking of sidebar.

I've checked both the Arabic 3.2.1 package and 3.3-trunk with Arabic language files, and the menu bar seems to reflect RTL direction correctly (see the screenshot).

Could you please confirm that you're using unmodified Twenty Eleven theme and not some child theme based on it? rtl-menus.gif doesn't look like default Twenty Eleven appearance to me.

#4 @SergeyBiryukov
9 years ago

  • Keywords reporter-feedback added

#5 @SergeyBiryukov
9 years ago

In case of a child theme, there was some discussion in #15863.

#6 @walid3
9 years ago

Thanks so much Sergey, your plugin solved the problem. yes it's a child theme, there should be some way of implementing that into the core as an option, like " use RTL css files for this theme " , yet, your plugin do that so well, thanks again.

#7 @walid3
9 years ago

  • Keywords reporter-feedback removed

#8 @SergeyBiryukov
9 years ago

  • Milestone Awaiting Review deleted
  • Resolution set to worksforme
  • Status changed from new to closed

The consensus in #15863 was not to include parent theme CSS by default, and a new option would contradict the "Decisions, not options" principle. So I guess the plugin is the only way to do it for now:
http://wordpress.org/extend/plugins/include-parent-theme-rtl-css/

Note: See TracTickets for help on using tickets.