#21281 closed defect (bug) (fixed)
RTL indention icons need to be flipped
| Reported by: | AlmogBaku | Owned by: | azaozz |
|---|---|---|---|
| Priority: | normal | Milestone: | 4.2 |
| Component: | TinyMCE | Version: | 3.4 |
| Severity: | major | Keywords: | has-patch |
| Cc: | Focuses: | rtl |
Description
By clicking the padding button(for indention) the action not works - it warpped the text by "p" element with padding-left insted or padding-right in RTL.
Attachments (2)
Change History (14)
#6
@
12 years ago
- Focuses rtl added
- Keywords needs-patch added
- Milestone Awaiting Review → Future Release
- Version 3.4.1 → 3.4
It looks like the icons just need to be flipped here.
#7
@
12 years ago
- Keywords has-patch added; needs-patch removed
- Milestone Future Release → 4.2
Also a few more questions...
- Do we need to flip the undo/redo icons for rtl?
- Maybe these also need to be flipped in
dashicons.css. Maybe also worth looking for other icons that might need to be flipped.
#8
@
12 years ago
- Summary RTL wysiwyg "padding"/"indention" action is not works as expected. → RTL indention icons need to be flipped
#9
@
12 years ago
Reference: @azaozz asked about the undo/redo in #polyglots:
https://wordpress.slack.com/archives/polyglots/p1421001395000007
#10
follow-up:
↓ 12
@
12 years ago
21281.2.patch looks good. Thinking it's better to open new ticket if we are going to switch the icons for undo/redo, and perhaps another one for any dashicons that need to be flipped for RTL.
Note:
See TracTickets
for help on using tickets.
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
Seems to be broken since 2.5 (2.3.3 used nested
<blockquote>tags for indentation).I've tried to add
directionality : "rtl"and directionality plugin to TinyMCE fiddle config, and couldn't make indentation work there either.