Opened 7 years ago

Closed 6 years ago

#6938 closed Bug (fixed)

IE/Opera: RTL language direction is not preserved when enter is pressed at the end of a list.

Reported by: James Owned by: Garry Yao
Priority: Normal Milestone: CKEditor 3.5.3
Component: Core : BiDi Version: 3.5
Keywords: IE Opera IBM Cc: Damian, joek, Satya Minnekanti

Description

Steps to reproduce the defect:

  1. Open the Ajax sample in IE (or Opera 11).
  2. Paste in the following code:
<p>
	paragraph 1</p>
<ol>
	<li>
		list item 1
		<ol>
			<li>
				list item 2
				<ol>
					<li>
						list item 3</li>
				</ol>
			</li>
		</ol>
	</li>
</ol>
  1. Press CTRL + A and click on RTL icon.
  2. Place the cursor at the end of the last list item.
  3. Press Enter 3 times.

Expected: The cursor should remain on the right and the language direction should be RTL.

Actual: The cursor is on the left and the language direction is be LTR.

Attachments (3)

language_direction_list_defect.jpg (29.7 KB) - added by James 7 years ago.
6938.patch (1.3 KB) - added by Garry Yao 7 years ago.
6938_tc.docx (13.8 KB) - added by Garry Yao 6 years ago.

Download all attachments as: .zip

Change History (11)

Changed 7 years ago by James

Changed 7 years ago by Garry Yao

Attachment: 6938.patch added

comment:1 Changed 7 years ago by Garry Yao

Keywords: HasPatch added
Status: newconfirmed

Browser irrelevant TC: press enter key at the following position:

<ol>
	<li dir="rtl">
		list item 1
		<ol>
			<li>^</li>
		</ol>
	</li>
</ol>

comment:2 Changed 6 years ago by Garry Yao

Keywords: HasPatch removed
Owner: set to Garry Yao
Status: confirmedreview

comment:3 Changed 6 years ago by Damian

Can we assign a milestone for this?

comment:4 Changed 6 years ago by Wiktor Walc

Milestone: CKEditor 3.5.3

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

Status: reviewreview_failed

In IE:

<ol dir="rtl">
	<li>
		list item 1
		<ol dir="ltr">
			<li>
				^</li>
		</ol>
	</li>
</ol>
  • Insert a few characters.
  • Hit ENTER once, insert a few more characters.
  • Hit ENTER twice. Note that the new list item is ltr.

Changed 6 years ago by Garry Yao

Attachment: 6938_tc.docx added

comment:6 Changed 6 years ago by Garry Yao

Status: review_failedreview

AFAIK that's just the desired behavior where language inherit from last list item (without considering list hierarchy ), it could be verified with the attached doc with MS Word.

comment:7 Changed 6 years ago by Frederico Caldeira Knabben

Status: reviewreview_passed

Agreed to follow the MS Word behavior for now.

comment:8 Changed 6 years ago by Garry Yao

Resolution: fixed
Status: review_passedclosed

Fixed with [6566].

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