Opened 6 years ago

Closed 6 years ago

Last modified 4 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 Owned by: Saare
Priority: Normal Milestone: CKEditor 3.4
Component: Core : Lists Version: 3.3
Keywords: IBM Cc: damo, 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 Saare 6 years ago.

Download all attachments as: .zip

Change History (10)

comment:1 Changed 6 years ago by fredck

  • Milestone set to CKEditor 3.4
  • Status changed from new to confirmed

comment:2 Changed 6 years ago by Saare

  • Owner set to Saare
  • Status changed from confirmed to assigned

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

Changed 6 years ago by Saare

comment:3 Changed 6 years ago by Saare

  • Status changed from assigned to review

comment:4 Changed 6 years ago by Saare

  • Component changed from General to Core : Lists

comment:5 Changed 6 years ago by fredck

  • Status changed from review to review_passed

comment:6 Changed 6 years ago by Saare

  • Resolution set to fixed
  • Status changed from review_passed to closed

Fixed with [5771].

comment:7 follow-up: Changed 6 years ago by damo

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 6 years ago by fredck

Replying to damo:

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

comment:9 Changed 4 years ago by fredck

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