Opened 13 years ago

Closed 13 years ago

Last modified 13 years ago

#6479 closed Bug (fixed)

BIDI: Language direction is not being preserved when pressing Enter after a paragraph format has been applied.

Reported by: James Owned by: Paweł Horzela
Priority: Normal Milestone: CKEditor 3.4.3
Component: UI : Enter Key Version: 3.4
Keywords: IBM Cc: Damian, joek, Satya Minnekanti

Description

Steps to reproduce the defect:

  1. Open the Ajax sample.
  2. Type a line of text in the editor.
  3. Apply Heading 1 paragraph format to the text.
  4. Click the RTL icon.
  5. Position the cursor at the end of the text & press Enter.

Expected: The cursor preserves the RTL language direction and appears on the right side of the editor.

Actual: The cursor now has LTR language direction and appears on the left side of the editor.

Attachments (3)

6479.patch (1.9 KB) - added by Paweł Horzela 13 years ago.
Patch
6479_2.patch (2.0 KB) - added by Paweł Horzela 13 years ago.
Patch
6479_3.patch (2.2 KB) - added by Paweł Horzela 13 years ago.
Patch

Download all attachments as: .zip

Change History (15)

comment:1 Changed 13 years ago by Garry Yao

Component: GeneralUI : Enter Key
Status: newconfirmed
Version: 3.4.13.4

comment:2 Changed 13 years ago by Frederico Caldeira Knabben

Milestone: CKeditor 3.4.3

comment:3 Changed 13 years ago by Paweł Horzela

Owner: set to Paweł Horzela
Status: confirmedassigned

Changed 13 years ago by Paweł Horzela

Attachment: 6479.patch added

Patch

comment:4 Changed 13 years ago by Paweł Horzela

Status: assignedreview

comment:5 Changed 13 years ago by Garry Yao

Status: reviewreview_failed

Note that styles other than the direction will not move as we're talking about existing from the headers in this case.

Changed 13 years ago by Paweł Horzela

Attachment: 6479_2.patch added

Patch

comment:6 Changed 13 years ago by Paweł Horzela

Status: review_failedreview

comment:7 Changed 13 years ago by Garry Yao

Status: reviewreview_failed

How about element::getDirection instead of hasAttribute( 'dir' ) ?

Changed 13 years ago by Paweł Horzela

Attachment: 6479_3.patch added

Patch

comment:8 Changed 13 years ago by Paweł Horzela

Status: review_failedreview

comment:9 Changed 13 years ago by Garry Yao

Status: reviewreview_passed

comment:10 Changed 13 years ago by Paweł Horzela

Resolution: fixed
Status: review_passedclosed

Fixed with [6078].

comment:11 Changed 13 years ago by Paweł Horzela

Reverted [6078]. Fixed with [6080].

comment:12 Changed 13 years ago by Paweł Horzela

Fixed with [6091].

Note: See TracTickets for help on using tickets.
© 2003 – 2022, CKSource sp. z o.o. sp.k. All rights reserved. | Terms of use | Privacy policy