Opened 7 years ago

Closed 6 years ago

#6861 closed Bug (fixed)

The Indent command changes a text direction

Reported by: Tobiasz Cudnik Owned by: Garry Yao
Priority: Normal Milestone: CKEditor 3.5.3
Component: Core : Styles Version: 3.4 Beta
Keywords: IBM Cc: jamcunni@…

Description

The Indent command changes a text direction.

  1. Use a following content with a selection:
    <ul>
    	<li dir="rtl">
    		1</li>
    	<li>
    		[2</li>
    	<li>
    		3]</li>
    </ul>
    
  2. Press the Increase Indent button.

Result:

<ul>
	<li dir="rtl">
		1
		<ul>
			<li>
				2</li>
			<li>
				3</li>
		</ul>
	</li>
</ul>

Expected: A new list element wrapping LIs 2 & 3 ?

Attachments (2)

6861.patch (850 bytes) - added by Garry Yao 6 years ago.
6861_2.patch (887 bytes) - added by Garry Yao 6 years ago.

Download all attachments as: .zip

Change History (11)

comment:1 Changed 7 years ago by James

Cc: jamcunni@… added
Keywords: IBM added

comment:2 Changed 6 years ago by Garry Yao

Component: GeneralCore : Styles
Status: newconfirmed
Version: 3.4 Beta

Changed 6 years ago by Garry Yao

Attachment: 6861.patch added

comment:3 Changed 6 years ago by Garry Yao

Owner: set to Garry Yao
Status: confirmedreview

comment:4 Changed 6 years ago by Damian

Can we target this for a milestone?

comment:5 Changed 6 years ago by Wiktor Walc

Milestone: CKEditor 3.5.3

comment:6 Changed 6 years ago by Sa'ar Zac Elias

Status: reviewreview_failed

In a RTL editor:

  • Use:
    <ul>
    	<li dir="ltr">
    		1</li>
    	<li>
    		[2</li>
    	<li>
    		3]</li>
    </ul>
    
  • Press "increase indent".

Note that the direction had changed for items 2 and 3.

Changed 6 years ago by Garry Yao

Attachment: 6861_2.patch added

comment:7 Changed 6 years ago by Garry Yao

Status: review_failedreview

comment:8 Changed 6 years ago by Sa'ar Zac Elias

Status: reviewreview_passed

comment:9 Changed 6 years ago by Garry Yao

Resolution: fixed
Status: review_passedclosed

Fixed with [6562].

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