Opened 5 years ago

Closed 4 years ago

Last modified 4 years ago

#8997 closed Bug (fixed)

Styles are lost on sub-lists when converting from numbered list to bulleted list and vice versa

Reported by: tmonahan Owned by: garry.yao
Priority: Normal Milestone: CKEditor 3.6.4
Component: Core : Lists Version: 3.3.2
Keywords: IBM Cc: damo, satya

Description

To Reproduce:

  • Copy the following content into the Source view of the editor. This is just a numbered list with 3 different levels. Each level has a font-size defined:
<ol style="font-size:20px;">
	<li>
		List 1</li>
	<li>
		List 1
		<ol style="font-size:16px;">
			<li>
				List 2</li>
			<li>
				List 2
				<ol style="font-size:12px;">
					<li>
						List3</li>
				</ol>
			</li>
		</ol>
	</li>
	<li>
		List 1</li>
</ol>

  • Return to wysiwyg mode and select the entire editor contents.
  • Click the Bulleted List icon in the toolbar to convert the list to a bulleted list.

Problem: The font styling for the sub-lists are lost. The font-size style for each sub-list is overwritten with the value from the top-level list:

<ul style="font-size: 20px;">
	<li>
		List 1</li>
	<li>
		List 1
		<ul style="font-size: 20px;">
			<li>
				List 2</li>
			<li>
				List 2
				<ul style="font-size: 20px;">
					<li>
						List3</li>
				</ul>
			</li>
		</ul>
	</li>
	<li>
		List 1</li>
</ul>

The styling for the sub-lists should be preserved. This also occurs when converting from bulleted lists to numbered lists.

Attachments (2)

8997.patch (1.4 KB) - added by garry.yao 4 years ago.
8997_2.patch (1.6 KB) - added by garry.yao 4 years ago.

Download all attachments as: .zip

Change History (11)

comment:1 Changed 5 years ago by j.swiderski

  • Component changed from General to Core : Lists
  • Status changed from new to confirmed
  • Version changed from 3.6.4 (SVN - trunk) to 3.3.2

This particular TC has been reproducible from CKEditor 3.3.2.

From CKE 3.0 till CKE 3.0.1 only style on top-level list was lost.

From CKE 3.0.2 till 3.3.1 all styles were lost.

comment:2 Changed 4 years ago by garry.yao

  • Milestone set to CKEditor 3.6.4
  • Owner set to garry.yao
  • Status changed from confirmed to review
Last edited 4 years ago by fredck (previous) (diff)

comment:3 Changed 4 years ago by fredck

  • Status changed from review to assigned

Did you eventually forgot to attach the patch? :)

Last edited 4 years ago by fredck (previous) (diff)

Changed 4 years ago by garry.yao

comment:4 Changed 4 years ago by garry.yao

  • Status changed from assigned to review

Oops, my fault.

comment:5 Changed 4 years ago by fredck

  • Status changed from review to review_failed

This test fails after patch:
http://ckeditor.t/tt/7290/1.html

Changed 4 years ago by garry.yao

comment:6 Changed 4 years ago by garry.yao

  • Status changed from review_failed to review

comment:7 Changed 4 years ago by fredck

  • Status changed from review to review_passed

comment:8 Changed 4 years ago by garry.yao

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

comment:9 Changed 4 years ago by garry.yao

Fixed with [7508].

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