#19042 closed defect (bug) (fixed)
RTL fixes for 3.3
| Reported by: |
|
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)
Change History (47)
comment:3
follow-up:
↓ 4
rosshanney — 19 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?
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).
comment:5
in reply to:
↑ 4
rosshanney — 19 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!
comment:6
SergeyBiryukov — 19 months ago
19042.help-and-screen-options.patch contains RTL adjustments for Help and Screen options since [19131].
SergeyBiryukov — 19 months ago
SergeyBiryukov — 19 months ago
comment:7
follow-up:
↓ 8
SergeyBiryukov — 19 months ago
19042.toggle-arrow-cleanup.patch cleans up unused .toggle-arrow class styles.
Being in RTL sections of colors-*.css files, they:
- Don't have .rtl prefix.
- Are overridden by editor-buttons.css.
Replying to SergeyBiryukov:
- 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 RTL in 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.
comment:10
azaozz — 19 months ago
In [19148]:
SergeyBiryukov — 19 months ago
19042.welcome-panel.patch contains RTL adjustments for Welcome Panel as per [19163].
rosshanney — 19 months ago
rosshanney — 19 months ago
comment:12
rosshanney — 19 months ago
The plugin update message on the plugins screen is a little out of place.
Please see 19042.plugin-update-message.diff.
rosshanney — 19 months ago
rosshanney — 19 months ago
comment:13
rosshanney — 19 months ago
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.
rosshanney — 19 months ago
rosshanney — 19 months ago
comment:14
rosshanney — 19 months ago
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
comment:15
azaozz — 18 months ago
In [19307]:
comment:16
helenyhou — 18 months ago
Submenu flyout jumps on focus: http://cl.ly/0v0a3C1v1M3d3z0E3w1Y
19042.flyoutjump.diff should fix it.
georgestephanis — 18 months ago
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!)
comment:19
ocean90 — 18 months ago
- Priority changed from normal to high
- Severity changed from normal to major
comment:20
ocean90 — 18 months ago
comment:21
ocean90 — 18 months ago
admin-menu-rtl.patch will fix all RTL issues with the admin menu, includes the issue mentioned by helenyhou in comment:16.
comment:22
azaozz — 18 months ago
In [19353]:
comment:23
ocean90 — 18 months ago
- Keywords needs-patch added; has-patch removed
- Priority changed from high to normal
- Severity changed from major to normal
comment:24
azaozz — 18 months ago
In [19360]:
comment:25
helenyhou — 18 months ago
Help styling is broken: http://cl.ly/2C331K2P2t2M0C0q2K0n
Working on a patch.
comment:26
helenyhou — 18 months ago
19042.helptabs.diff fixes help styling, probably broken in r19361.
comment:27
azaozz — 18 months ago
In [19371]:
comment:28
ocean90 — 18 months ago
- Keywords needs-patch removed
Please open new tickets for still existing RTL issues.
comment:29
ocean90 — 18 months ago
- Resolution set to fixed
- Status changed from new to closed
comment:30
azaozz — 18 months ago
In [19446]:






In [19080]: