Ticket #8085 (closed Bug: invalid)

Opened 3 years ago

Last modified 3 years ago

language direction applied to list item removed after indentation

Reported by: satya Owned by:
Priority: Normal Milestone:
Component: General Version: 3.6.1
Keywords: IBM Cc: damo, tmonahan

Description

paste the following code in to source view.

 <ul>
	<li>
		List item 1 (LTR)</li>
	<li dir="rtl">
		List item 2 (RTL)</li>
	<li>
		List item 3 (LTR)</li>
</ul>

Go back to rich text, keep the cursor in 2nd list item and click on Increase Indent icon.

Expected Result A sub list is created with RTL Lang direction and language direction(dir="rtl") applied to 2nd list item remains.

Actual Result A sub list is created with RTL Lang direction but language direction(dir="rtl") applied to 2nd list item is removed.

Change History

comment:1 follow-up: ↓ 2 Changed 3 years ago by j.swiderski

  • Status changed from new to pending

The produced result is:

<ul>
	<li>
		List item 1 (LTR)
		<ul dir="rtl">
			<li>
				List item 2 (RTL)</li>
		</ul>
	</li>
	<li>
		List item 3 (LTR)</li>
</ul>

As You have noticed the RTL is applied to the main list element (ul). Why do you want to apply RTL direction to its children if they inherit this setting. In my opinion this would be extra and unnecessary attribute.

Is there some particular TC that makes my point of view wrong?

comment:2 in reply to: ↑ 1 Changed 3 years ago by satya

Replying to j.swiderski:

The produced result is:

<ul>
	<li>
		List item 1 (LTR)
		<ul dir="rtl">
			<li>
				List item 2 (RTL)</li>
		</ul>
	</li>
	<li>
		List item 3 (LTR)</li>
</ul>

As You have noticed the RTL is applied to the main list element (ul). Why do you want to apply RTL direction to its children if they inherit this setting. In my opinion this would be extra and unnecessary attribute.

Is there some particular TC that makes my point of view wrong?

@j.swiderski until yesterday the language direction(dir="rtl") applied to list item remains i'm seeing this new behavior only after i merged the latest changes from last night. So i thought this has happened bcoz of a regression from another fix.

comment:3 follow-up: ↓ 4 Changed 3 years ago by j.swiderski

@garry.yao has described and changed it in #8042.

For me it's appropriate behavior. Do you agree and we can close this issue or perhaps there is some special case which makes #8042 invalid?

comment:4 in reply to: ↑ 3 Changed 3 years ago by satya

Replying to j.swiderski:

@garry.yao has described and changed it in #8042.

For me it's appropriate behavior. Do you agree and we can close this issue or perhaps there is some special case which makes #8042 invalid?

@j.swiderski i agree, please close this defect.

comment:5 Changed 3 years ago by j.swiderski

  • Status changed from pending to closed
  • Resolution set to invalid
Note: See TracTickets for help on using tickets.
© 2003 – 2012 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy