Opened 19 months ago

Closed 18 months ago

Last modified 18 months ago

#19042 closed defect (bug) (fixed)

RTL fixes for 3.3

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

Description (last modified by yoavf)

It's time for a main 3.3 RTL ticket - I know it's usually better to split things up (see {38} ) but I think that at this point it's better to consolidate issues so we can all work on this together.

The attached patch refreshes admin-bar-rtl completely to account for 3.3 changes, fixes a bug in the uploader RTL css, and with the admin footer.

The patch also consolidates fixes for:

Attachments (17)

3.3.rtl-1.patch (8.5 KB) - added by yoavf 19 months ago.
bubble.jpg (10.3 KB) - added by azaozz 19 months ago.
19042.help-and-screen-options.patch (3.9 KB) - added by SergeyBiryukov 19 months ago.
19042.toggle-arrow-cleanup.patch (1.6 KB) - added by SergeyBiryukov 19 months ago.
19042.welcome-panel.patch (1.3 KB) - added by SergeyBiryukov 19 months ago.
19042.plugin-update-message.diff (463 bytes) - added by rosshanney 19 months ago.
plugin-update-message.png (7.9 KB) - added by rosshanney 19 months ago.
19042.collapse-icon-and-submenu-shadows.diff (1.5 KB) - added by rosshanney 19 months ago.
collapse-icon-and-submenu-shadows.png (8.3 KB) - added by rosshanney 19 months ago.
19042.admin-bar-howdy-menu.diff (1.4 KB) - added by rosshanney 19 months ago.
admin-bar-howdy-menu.png (2.7 KB) - added by rosshanney 19 months ago.
19042.admin-bar-howdy-menu-2.diff (1.4 KB) - added by rosshanney 19 months ago.
Update for "Howdy" menu patch, after [19261] changes.
19042.admin-bar-howdy-menu-3.diff (1.5 KB) - added by rosshanney 18 months ago.
Another update for "Howdy" patch, after [19270].
19042.flyoutjump.diff (389 bytes) - added by helenyhou 18 months ago.
Fix for flyout jumpiness
plugin-deactivated-bad-period.png (180.7 KB) - added by georgestephanis 18 months ago.
When deactivating a plugin (and I'm not sure where else) the text alert reads .Plugin deactivated (sic)
admin-menu-rtl.patch (7.3 KB) - added by ocean90 18 months ago.
19042.helptabs.diff (786 bytes) - added by helenyhou 18 months ago.
Help tab

Download all attachments as: .zip

Change History (47)

yoavf19 months ago

  • Description modified (diff)

In [19080]:

RTL fixes for 3.3, props yoavf SergeyBiryukov rosshanney, see #19042 fixes #18000, fixes #17988, fixes #19006

comment:3 follow-up: ↓ 4   rosshanney19 months ago

I've noticed a few RTL issues with the admin bar in Internet Explorer. For example, in IE9, the white update count circle doesn't appear correctly and the little downward pointing arrows don't appear at all (or appear in the wrong place with the patch proposed in #19086).

Also, in IE7, everything in the admin bar disappears altogether on hover.

Is it best to create separate tickets for these issues, or just discuss them here?

comment:4 in reply to: ↑ 3 ; follow-up: ↓ 5   azaozz19 months ago

Replying to rosshanney:

The bubble looks fine to me:


and the arrows are most likely going to be removed (they were meant to appear on hover if the menus needed clicks to open).

Last edited 19 months ago by azaozz (previous) (diff)

comment:5 in reply to: ↑ 4   rosshanney19 months ago

I'm referring to the available update count notification (the white circle), not the comments bubble.

Replying to azaozz:

and the arrows are most likely going to be removed (they were meant to appear on hover if the menus needed clicks to open).

Ah, I see. Fair enough!

19042.help-and-screen-options.patch contains RTL adjustments for Help and Screen options since [19131].

Last edited 19 months ago by SergeyBiryukov (previous) (diff)

comment:7 follow-up: ↓ 8   SergeyBiryukov19 months ago

19042.toggle-arrow-cleanup.patch cleans up unused .toggle-arrow class styles.

Being in RTL sections of colors-*.css files, they:

  1. Don't have .rtl prefix.
  2. Are overridden by editor-buttons.css.

comment:8 in reply to: ↑ 7   azaozz19 months ago

Replying to SergeyBiryukov:

  1. Are overridden by editor-buttons.css.

Editor-buttons is supposed to hold all styling for outside the editors (mostly buttons and toolbars) as they can be used on the front end now. For the blue theme and specific RTL for the admin we could use a bit more specific selectors, i.e. prefix the selectors by one of the container classes/IDs like #wpcontent, #wpbody, .wrap, etc. as editor-buttons.css is loaded after the RTL stylesheets (mid-page).

Last edited 19 months ago by azaozz (previous) (diff)

In [19147]:

Help and Screen Options RTL adjustments, props SergeyBiryukov, see #19042

In [19148]:

Remove unused .toggle-arrow styling, props SergeyBiryukov, see #19042

19042.welcome-panel.patch contains RTL adjustments for Welcome Panel as per [19163].

The plugin update message on the plugins screen is a little out of place.


Please see 19042.plugin-update-message.diff.

Last edited 19 months ago by rosshanney (previous) (diff)

The collapse icon points the wrong way when the admin menu is collapsed (.folded). Also, the shadows on admin menu flyouts don't seem quite right.


Please see 19042.collapse-icon-and-submenu-shadows.diff

The "Howdy" section of the admin bar no longer appears at the far left. I think caused by [19230].


Please see 19042.admin-bar-howdy-menu.diff

Update for "Howdy" menu patch, after [19261] changes.

Another update for "Howdy" patch, after [19270].

In [19307]:

More RTL fixes, props rosshanney, see #19042

Fix for flyout jumpiness

Submenu flyout jumps on focus: http://cl.ly/0v0a3C1v1M3d3z0E3w1Y

19042.flyoutjump.diff should fix it.

When deactivating a plugin (and I'm not sure where else) the text alert reads .Plugin deactivated (sic)

Ignore my last bug report-image, I'm an idiot newb to RTL.

That being said, my latest patch on #18654 -- 18654.with-201-percent-more-rtl.diff -- also happens to fix the user menu in RTL. I'm cross-posting so it doesn't get lost in the shuffle somehow (thanks, jjj!)

  • Priority changed from normal to high
  • Severity changed from normal to major

Admin bar needs some RTL love.

http://cl.ly/BvCC/Bildschirmfoto%202011-11-18%20um%2013.24.03.png

Last edited 18 months ago by ocean90 (previous) (diff)

admin-menu-rtl.patch will fix all RTL issues with the admin menu, includes the issue mentioned by helenyhou in comment:16.

In [19353]:

More RTL for the admin menu, props ocean90, see#19042

  • Keywords needs-patch added; has-patch removed
  • Priority changed from high to normal
  • Severity changed from major to normal

http://cl.ly/ByjA/Bildschirmfoto%202011-11-19%20um%2018.52.30.png

The progress bar shows a dark line at beginning. Only in thickbox media uploader.

In [19360]:

Remove old styling for rtl progress bar, see #19042

Help styling is broken: http://cl.ly/2C331K2P2t2M0C0q2K0n

Working on a patch.

Help tab

19042.helptabs.diff fixes help styling, probably broken in r19361.

In [19371]:

Help tabs RTL fixes, props helenyhou, see #19042

  • Keywords needs-patch removed

Please open new tickets for still existing RTL issues.

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

In [19446]:

Few more RTL and IE7 fixes, see #19042

Note: See TracTickets for help on using tickets.