Opened 6 years ago

Closed 5 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 c Owned by: garry.yao
Priority: Normal Milestone: CKEditor 3.5.3
Component: Core : BiDi Version: 3.5
Keywords: IE Opera IBM Cc: damo, joek, satya

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 c 6 years ago.
6938.patch (1.3 KB) - added by garry.yao 6 years ago.
6938_tc.docx (13.8 KB) - added by garry.yao 5 years ago.

Download all attachments as: .zip

Change History (11)

Changed 6 years ago by james c

Changed 6 years ago by garry.yao

comment:1 Changed 6 years ago by garry.yao

  • Keywords HasPatch added
  • Status changed from new to confirmed

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 changed from confirmed to review

comment:3 Changed 5 years ago by damo

Can we assign a milestone for this?

comment:4 Changed 5 years ago by wwalc

  • Milestone set to CKEditor 3.5.3

comment:5 Changed 5 years ago by Saare

  • Status changed from review to review_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 5 years ago by garry.yao

comment:6 Changed 5 years ago by garry.yao

  • Status changed from review_failed to review

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 5 years ago by fredck

  • Status changed from review to review_passed

Agreed to follow the MS Word behavior for now.

comment:8 Changed 5 years ago by garry.yao

  • Resolution set to fixed
  • Status changed from review_passed to closed

Fixed with [6566].

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