Opened 6 years ago

Closed 5 years ago

#10890 closed Bug (fixed)

Error thrown when we press delete after first list item in a Numbered/Bulleted list inside a table

Reported by: Satya Minnekanti Owned by: Olek Nowodziński
Priority: Normal Milestone: CKEditor 4.3.1
Component: General Version: 3.6.4
Keywords: IBM Cc: Teresa Monahan, Irina

Description

To reproduce the defect:

  1. Open any sample and insert a table.
  1. Keep cursor inside a table cell & click on Numbered/Bulleted list icon in the toolbar to start a list
  1. New list created and cursor stays after first list item in Numbered/Bulleted list
  1. Press Delete

Expected Result: Nothing should happen

Actual Result: In IE A javascript error thrown & error is Unable to get the value of the property is: object is null or undefined.

In Firefox list is deleted

Change History (6)

comment:1 Changed 6 years ago by Jakub Ś

Status: newconfirmed
Version: 3.6.4

I actually got error in every Browser. Chrome and IE don't remove bullet while Firefox does but every browser throws error:

Message: TypeError: nextBlock is null
Code: if ( nextBlock.is( 'li' ) ) {
URI: /ckeditor4/plugins/list/plugin.js Line: 721

Problem can be reproduced from CKEditor 3.6.4 rev. [7540]

comment:2 Changed 5 years ago by Jakub Ś

Summary: IE: Error thrown when we press delete after first list item in a Numbered/Bulleted list inside a tableError thrown when we press delete after first list item in a Numbered/Bulleted list inside a table

#11128 was marked as duplicate.

Add the table with the list inside:

<table border="1" cellpadding="1" cellspacing="1" style="width:500px">
	<tbody>
		<tr>
			<td>
			<ol>
				<li>One</li>
				<li>Two</li>
			</ol>
			</td>
			<td>Test</td>
		</tr>
	</tbody>
</table>

Move cursor to the end of the second item of the list and press Delete.

comment:3 Changed 5 years ago by Olek Nowodziński

Owner: set to Olek Nowodziński
Status: confirmedassigned

comment:4 Changed 5 years ago by Olek Nowodziński

Status: assignedreview

Pushed fix to t/10890.

comment:5 Changed 5 years ago by Frederico Caldeira Knabben

Milestone: CKEditor 4.3.1
Status: reviewreview_passed

comment:6 Changed 5 years ago by Olek Nowodziński

Resolution: fixed
Status: review_passedclosed

git:58b6b95 landed in master.

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