Make WordPress Core

Opened 14 years ago

Closed 14 years ago

Last modified 14 years ago

#23101 closed defect (bug) (fixed)

RTL on visual editor

Reported by: ramiy Owned by: nacin
Priority: normal Milestone: 3.5.1
Component: RTL Version: 3.5
Severity: normal Keywords: has-patch commit
Cc: Focuses:

Description

check out the screenshot.

Attachments (4)

visual-editor-rtl.png (86.9 KB ) - added by ramiy 14 years ago.
format-dropdown-wp34.png (13.1 KB ) - added by SergeyBiryukov 14 years ago.
23101.diff (346 bytes ) - added by helen 14 years ago.
rtl-tinymce-dropdowns.png (16.4 KB ) - added by helen 14 years ago.

Download all attachments as: .zip

Change History (13)

#1 follow-up: @SergeyBiryukov
14 years ago

Confirmed. Looks better in 3.4: format-dropdown-wp34.png.

I would expect the dropdowns in RTL to be right-aligned with the arrow though.

#2 @SergeyBiryukov
14 years ago

  • Milestone Awaiting Review3.5.1

#3 in reply to: ↑ 1 @ramiy
14 years ago

Replying to SergeyBiryukov:

Looks better in 3.4

Sergey, they both look bad from user experience perspective. The dropdown should be wider and higher.

@helen
14 years ago

#4 @helen
14 years ago

  • Keywords has-patch added; needs-patch removed

I think the only thing we can do for 3.5.1 is move the scrollbar back over and remove that lame -14px margin thing that I assume was a hack to make it look like it was aligned to the right side, which only worked with that dropdown with only default items: 23101.diff

It'll align with the left side of the toolbar item, and the selection icon is on the left, but I don't know what we can do about that positioning - it's all set by TinyMCE and there are no RTL icons. Maybe can investigate further in 3.6, but for 3.5.1 I'm not sure we can/should do too much more.

ramiy - can you explain what you mean by the dropdown being higher? And also, in your screenshot, could you explain what the arrow on the left is for? Screenshots are helpful, but please try to be descriptive with words where you can.

#5 @nacin
14 years ago

  • Keywords commit added

If 23101.diff restores things to the 3.4 screenshot (attachment:format-dropdown-wp34.png), I'm quite happy for this for 3.5.1.

#6 @helen
14 years ago

Right, yes, screenshots. It does, although it will be aligned to the left side of the box rather than seemingly-randomly 14 pixels over. We can restore that, but it's pretty meaningless. Image shows the default Format picker on the left, with added dropdowns and font size picker showing on the right.

http://core.trac.wordpress.org/raw-attachment/ticket/23101/rtl-tinymce-dropdowns.png

We should probably figure out whether or not we can make TinyMCE behave better for RTL in 3.6. I have a suspicion it's not us.

#7 @nacin
14 years ago

In 23275:

Restore TinyMCE dropdowns to their 3.4 appearance in RTL. props helen. see #23101.

#8 @nacin
14 years ago

  • Owner set to nacin
  • Resolutionfixed
  • Status newclosed

In 23276:

Restore TinyMCE dropdowns to their 3.4 appearance in RTL.

Merges [23275] to the 3.5 branch.
props helen.
fixes #23101.

#9 @nacin
14 years ago

Let's do a new ticket for improving the styles and working with TinyMCE upstream (as necessary). Considering this ticket fixed, as the regression is handled.

Note: See TracTickets for help on using tickets.