Was fighting with the order of secondary tabs made created from Panels and figured out that the Seven theme in Drupal 7 is showing them reversed. They are displayed correct in other themes like Bartik and Rubik. They are displayed fine in RTL though.

Checked the CSS in Firebug and it looks like the "float: right" on "ul.secondary li" is causing the problem.

ul.secondary {
  float: right; /* LTR */
  font-size: 0.923em;
  padding: 0 3px 5px;
  line-height: 1.385em;
  overflow: hidden;
  background-color: #fff;
ul.secondary li {
  margin: 0 5px;
  float: right; /* LTR */

It's working fine in both LTR and RTL when "float: right" is removed or set to none. Not sure which one is the correct CSS approach.

Attached a D8 patch setting float to none.

#4 secondary-tabs-float-7.patch328 bytesbeltofte
PASSED: [[SimpleTest]]: [MySQL] 36,608 pass(es). View
secondary-tabs-float.patch328 bytesbeltofte
PASSED: [[SimpleTest]]: [MySQL] 33,302 pass(es). View
Members fund testing for the Drupal project. Drupal Association Learn more


ParisLiakos’s picture

I thought at first it was menus system bug,but after switching to bartik and other themes menu order was fine!

ParisLiakos’s picture

Status: Active » Needs review
ParisLiakos’s picture

Status: Needs review » Reviewed & tested by the community
Issue tags: +needs backport to D7

i tried the patch and works.Testbot is also happy:)

beltofte’s picture

328 bytes
PASSED: [[SimpleTest]]: [MySQL] 36,608 pass(es). View

Attached patch for Drupal 7.

beltofte’s picture

Hmm..... okay, looks like I should wait attaching a patch for Drupal 7 till the issue is changed to version 7. Testbot might not like the D7 patch on D8.

Dries’s picture

Version: 8.x-dev » 7.x-dev

Committed to 8.x. Moving to 7.x for webchick.

beltofte’s picture

#4: secondary-tabs-float-7.patch queued for re-testing.

webchick’s picture

Status: Reviewed & tested by the community » Fixed

Committed and pushed to 7.x. Thanks!

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