Opened 6 years ago

Closed 6 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 6 years ago.

Download all attachments as: .zip

Change History (12)

comment:1 Changed 6 years ago by Krzysztof Studnik

Status: newconfirmed
Version: 3.43.0

comment:2 Changed 6 years ago by Garry Yao

Component: Core : BiDiCore : Lists

comment:3 Changed 6 years ago by Satya Minnekanti

Cc: satya_minnekanti@… added
Keywords: IBM added

comment:4 Changed 6 years ago by Garry Yao

Merge list logic should now consider text direction.

Changed 6 years ago by Garry Yao

Attachment: 7448.patch added

comment:5 Changed 6 years ago by Garry Yao

Owner: set to Garry Yao
Status: confirmedreview

comment:6 Changed 6 years ago by Garry Yao

Ticket tc added with [7128].

comment:7 Changed 6 years ago by Garry Yao

Ticket Test: run OR view source.

Last edited 6 years ago by Garry Yao (previous) (diff)

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

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

TC issue fixed.

comment:10 Changed 6 years ago by Frederico Caldeira Knabben

Milestone: CKEditor 3.6.2
Status: reviewreview_passed

comment:11 Changed 6 years ago by Garry Yao

Resolution: fixed
Status: review_passedclosed

Fixed with [7210].

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