Opened 6 years ago

Closed 6 years ago

Last modified 5 years ago

#7657 closed Bug (fixed)

BIDI: The wrong indentation is applied when converting indented RTL paragraphs into lists

Reported by: jamescun Owned by: garry.yao
Priority: Normal Milestone: CKEditor 3.6.1
Component: Core : BiDi Version: 3.5.3
Keywords: IBM Cc: damo, satya, tmonahan

Description

Steps to reproduce the defect:

  1. Open the Ajax sample.
  1. Type 3 paragraphs of text.
  1. Indent the 2nd paragraph once.
  1. Indent the 3rd paragraph twice.
  1. Select all 3 paragraphs & click the RTL button in the toolbar.
  1. Selcet all 3 paragraphs & click the numbers/bullets button in the toolbar.

Expected: A list should be created with the 2nd list item having a margin right of 40px & the 3rd list item having a margin right of 80px.

Actual: A list is created with the 2nd list item having a margin left of 40px & the 3rd list item having a margin left of 80px. The wrong indentation is applied which causes the list to lose all indentation.

Attachments (1)

7657.patch (858 bytes) - added by garry.yao 6 years ago.

Download all attachments as: .zip

Change History (7)

comment:1 Changed 6 years ago by satya

  • Version changed from 3.5.4 (SVN - trunk) to 3.5.3

This is a regression in 3.5.3. it was working since 3.5.2

comment:2 Changed 6 years ago by j.swiderski

  • Status changed from new to confirmed

What @jamescun has described, has started from revision [6551]

But this feature has stopped working properly form revision [6478]

From 6478 to 6551 when applying numbered/bulleted list to selected text it either had no effect or removed the whole list.

Changed 6 years ago by garry.yao

comment:3 Changed 6 years ago by garry.yao

  • Keywords BIDI removed
  • Owner set to garry.yao
  • Status changed from confirmed to review

comment:4 Changed 6 years ago by fredck

  • Milestone set to CKEditor 3.6.1
  • Status changed from review to review_passed

comment:5 Changed 6 years ago by garry.yao

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

Fixed with [6911].

Note: See TracTickets for help on using tickets.
© 2003 – 2016 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy