Opened 7 years ago

Closed 7 years ago

Last modified 5 years ago

#6059 closed Bug (fixed)

BIDI: switching from Numbered to Bulleted List & vice versa is changing Lang direction of the list

Reported by: Satya Minnekanti Owned by: Sa'ar Zac Elias
Priority: Normal Milestone: CKEditor 3.4
Component: Core : Lists Version: 3.3
Keywords: IBM Cc: Damian, joek

Description

To reproduce the defect:

  1. Open Ajax sample.
  1. Type few lines of text, select all the lines and Click on RTL icon in the Tool bar.
  1. See that all the lines of text is moved to right,RTL and Align Left icons are selected in the Tool bar.
  1. Select all the lines of text and click on Insert/Remove Numbered list icon in the Tool bar.
  1. See that a Numbered list appears with RTL language direction with the selected lines.
  1. Now select all the items in Numbered list and click on Insert/Remove Bulleted list icon in the Tool bar.

Expected Result:

Numbered list disappears and a Bulleted list appears with RTL language direction with the selected lines.

Actual Result:

Numbered list disappears and a Bulleted list appears with LTR language direction instead of RTL Language direction

same behavior happens when we make a Bulleted List first and then switch back to Numbered List.

Attachments (1)

6059.patch (872 bytes) - added by Sa'ar Zac Elias 7 years ago.

Download all attachments as: .zip

Change History (10)

comment:1 Changed 7 years ago by Frederico Caldeira Knabben

Milestone: CKEditor 3.4
Status: newconfirmed

comment:2 Changed 7 years ago by Sa'ar Zac Elias

Owner: set to Sa'ar Zac Elias
Status: confirmedassigned

The problem here is wider, attributes are not preserved when changing list type.

Changed 7 years ago by Sa'ar Zac Elias

Attachment: 6059.patch added

comment:3 Changed 7 years ago by Sa'ar Zac Elias

Status: assignedreview

comment:4 Changed 7 years ago by Sa'ar Zac Elias

Component: GeneralCore : Lists

comment:5 Changed 7 years ago by Frederico Caldeira Knabben

Status: reviewreview_passed

comment:6 Changed 7 years ago by Sa'ar Zac Elias

Resolution: fixed
Status: review_passedclosed

Fixed with [5771].

comment:7 Changed 7 years ago by Damian

This was checked-in into the trunk but should also be in 3.4 branch to verify that it works.

comment:8 in reply to:  7 Changed 7 years ago by Frederico Caldeira Knabben

Replying to damo:

All recent trunk changes have just been merged into the 3.4.x.

comment:9 Changed 5 years ago by Frederico Caldeira Knabben

Cc: damo,joekdamo, joek
Note: See TracTickets for help on using tickets.
© 2003 – 2017 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy