Opened 13 years ago

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

Download all attachments as: .zip

Change History (12)

comment:1 Changed 13 years ago by Krzysztof Studnik

Status: newconfirmed
Version: 3.43.0

comment:2 Changed 13 years ago by Garry Yao

Component: Core : BiDiCore : Lists

comment:3 Changed 13 years ago by Satya Minnekanti

Cc: satya_minnekanti@… added
Keywords: IBM added

comment:4 Changed 13 years ago by Garry Yao

Merge list logic should now consider text direction.

Changed 13 years ago by Garry Yao

Attachment: 7448.patch added

comment:5 Changed 13 years ago by Garry Yao

Owner: set to Garry Yao
Status: confirmedreview

comment:6 Changed 13 years ago by Garry Yao

Ticket tc added with [7128].

comment:7 Changed 13 years ago by Garry Yao

Ticket Test: run OR view source.

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

comment:8 in reply to:  7 Changed 13 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 13 years ago by Garry Yao

TC issue fixed.

comment:10 Changed 13 years ago by Frederico Caldeira Knabben

Milestone: CKEditor 3.6.2
Status: reviewreview_passed

comment:11 Changed 13 years ago by Garry Yao

Resolution: fixed
Status: review_passedclosed

Fixed with [7210].

Note: See TracTickets for help on using tickets.
© 2003 – 2022, CKSource sp. z o.o. sp.k. All rights reserved. | Terms of use | Privacy policy