Opened 6 years ago

Last modified 4 years ago

#11065 confirmed Bug

Incorrect paragraph style after applying indentation and changing text direction

Reported by: Irina Owned by:
Priority: Normal Milestone:
Component: General Version: 4.2.2
Keywords: IBM Cc: Teresa Monahan, Satya Minnekanti, peter

Description

  1. Load the attached sample page
  2. Click into the 1st inline editor
  3. Ctrl+A to select all text
  4. Click the button to change the text direction from left to right
  5. Click increase indent button
  6. Click the button to change the text direction from right to left
  7. Check the paragraph style

Expected result: margin-left: 40px;

Actual result: margin-right: 40px;

This only happens when we have two editors on the page.

Attachments (1)

sample.html (663 bytes) - added by Irina 6 years ago.

Download all attachments as: .zip

Change History (4)

Changed 6 years ago by Irina

Attachment: sample.html added

comment:1 Changed 5 years ago by Piotrek Koszuliński

Status: newconfirmed

Confirmed on Chrome. Didn't check other browsers, but I guess it's a cross-browser problem. And it really requires two editors what makes it very interesting :).

Reproducible from at least from 4.0, and only on inline editor.

comment:2 Changed 4 years ago by Piotrek Koszuliński

  • The editor#dirChanged events is fired on both editors when changing the text direction in the first one. Most likely this is a cause of this issue.
  • This may turn out to be a tricky issue due to the bidi’s weird implementation. However, first thing to fix will be the event. It may also fix the errors which are thrown. If not, then it’d be good to check how is the range created to fix those errors.

comment:3 Changed 4 years ago by Anna Tomanek

Summary: Incorrect paragraph style after applying indentation and changing RTL/LTR directionIncorrect paragraph style after applying indentation and changing text direction
Note: See TracTickets for help on using tickets.
© 2003 – 2019 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy