Opened 6 years ago

Closed 5 years ago

#7448 closed Bug (fixed)

Create list on RTL paragraph

Reported by: garry.yao Owned by: garry.yao
Priority: Normal Milestone: CKEditor 3.6.2
Component: Core : Lists Version: 3.0
Keywords: IBM Cc: satya_minnekanti@…

Description

  1. Load editor with the following source and selection:
    <ol>
    	<li>
    		ltr</li>
    </ol>
    <p dir="rtl">
    rtl^</p>
    
  2. Click "Numbered List" button;
  • Actual Result:
    <ol dir="rtl">
    	<li>
    		ltr</li>
    	<li>
    		rtl</li>
    </ol>
    
  • Expected:
    <ol>
    	<li>
    		ltr</li>
    	<li dir="rtl">
    		rtl</li>
    </ol>
    

Attachments (1)

7448.patch (1.2 KB) - added by garry.yao 5 years ago.

Download all attachments as: .zip

Change History (12)

comment:1 Changed 6 years ago by krst

  • Status changed from new to confirmed
  • Version changed from 3.4 to 3.0

comment:2 Changed 5 years ago by garry.yao

  • Component changed from Core : BiDi to Core : Lists

comment:3 Changed 5 years ago by satya

  • Cc satya_minnekanti@… added
  • Keywords IBM added

comment:4 Changed 5 years ago by garry.yao

Merge list logic should now consider text direction.

Changed 5 years ago by garry.yao

comment:5 Changed 5 years ago by garry.yao

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

comment:6 Changed 5 years ago by garry.yao

Ticket tc added with [7128].

comment:7 follow-up: Changed 5 years ago by garry.yao

Ticket Test: run OR view source.

Last edited 5 years ago by garry.yao (previous) (diff)

comment:8 in reply to: ↑ 7 Changed 5 years ago by fredck

Replying to garry.yao:

Ticket Test: run OR view source.

This test is currently PASSing on FF without patch, while the original TC can be confirmed manual. It makes it not reliable and must be verified.

comment:9 Changed 5 years ago by garry.yao

TC issue fixed.

comment:10 Changed 5 years ago by fredck

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

comment:11 Changed 5 years ago by garry.yao

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

Fixed with [7210].

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