Opened 13 years ago

Closed 13 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 13 years ago.
6938.patch (1.3 KB) - added by Garry Yao 13 years ago.
6938_tc.docx (13.8 KB) - added by Garry Yao 13 years ago.

Download all attachments as: .zip

Change History (11)

Changed 13 years ago by James

Changed 13 years ago by Garry Yao

Attachment: 6938.patch added

comment:1 Changed 13 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 13 years ago by Garry Yao

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

comment:3 Changed 13 years ago by Damian

Can we assign a milestone for this?

comment:4 Changed 13 years ago by Wiktor Walc

Milestone: CKEditor 3.5.3

comment:5 Changed 13 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 13 years ago by Garry Yao

Attachment: 6938_tc.docx added

comment:6 Changed 13 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 13 years ago by Frederico Caldeira Knabben

Status: reviewreview_passed

Agreed to follow the MS Word behavior for now.

comment:8 Changed 13 years ago by Garry Yao

Resolution: fixed
Status: review_passedclosed

Fixed with [6566].

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