Make WordPress Core

Opened 13 years ago

Closed 13 years ago

Last modified 13 years ago

#18029 closed defect (bug) (fixed)

Collapse menu in RTL mode

Reported by: ramiy's profile ramiy Owned by: yoavf's profile yoavf
Milestone: 3.2.1 Priority: normal
Severity: normal Version: 3.2
Component: RTL Keywords: has-patch
Focuses: Cc:

Description

When collapsing the side menu on wordpress dashboard, the content area should become wider.

It's not working on RTL dashboards, the content area is not getting wider.

Attachments (2)

18029.diff (372 bytes) - added by rosshanney 13 years ago.
18029.2.diff (352 bytes) - added by rosshanney 13 years ago.

Download all attachments as: .zip

Change History (16)

#1 @nacin
13 years ago

  • Milestone changed from Awaiting Review to 3.2.1

#2 @ramiy
13 years ago

  • Summary changed from Collapse menu in RTL more to Collapse menu in RTL mode

@rosshanney
13 years ago

#3 @rosshanney
13 years ago

  • Keywords has-patch added; needs-patch removed

@rosshanney
13 years ago

#4 @rosshanney
13 years ago

I believe new patch is sufficient, as admin menu cannot be collapsed without JavaScript.

#5 @ramiy
13 years ago

patch tested, not working.

#6 @rosshanney
13 years ago

Are you sure? It seems to do the trick for me. Do you have SCRIPT_DEBUG enabled (the patch is for global-rtl.dev.css, not global-rtl.css)?

#7 @ramiy
13 years ago

i tested the patch again (18029.2.diff - global-rtl.dev.css) in a different site. still not working.

#8 @nacin
13 years ago

Patch works for me. The issue is that we need to properly override the CSS changes that went into [18349].

#9 @nacin
13 years ago

In [18420]:

Fix RTL admin menu collapsing margin. props rosshanney, see #18029 for trunk.

#10 @nacin
13 years ago

In [18421]:

Fix RTL admin menu collapsing margin. props rosshanney, see #18029 for 3.2.

#11 @nacin
13 years ago

  • Severity changed from major to normal

Leaving open for final review by azaozz. Some other things in global-rtl.css might deserve some edits.

#12 @azaozz
13 years ago

  • Resolution set to fixed
  • Status changed from new to closed

In [18428]:

Fix expanding the admin menu in RTL mode when no JS and collapsing it when JS, fixes #18029 for trunk

#13 @azaozz
13 years ago

In [18429]:

Fix expanding the admin menu in RTL mode when no JS and collapsing it when JS, fixes #18029 for 3.2

#14 @SergeyBiryukov
13 years ago

  • Keywords rtl-feedback removed
Note: See TracTickets for help on using tickets.