Opened 13 years ago

Closed 13 years ago

Last modified 13 years ago

#8249 closed Bug (fixed)

[FF]: Problems with deleting last list item

Reported by: Jakub Ś Owned by: Garry Yao
Priority: Normal Milestone: CKEditor 3.6.3
Component: Core : Lists Version: 3.0
Keywords: IBM Cc: satya_minnekanti@…

Description (last modified by Jakub Ś)

I have found this issue when reproducing 8248

Scenario 1

  1. Paste the following code:
    <ol>
    	<li> FirstListItem  </li>
    	<li> SecondListItem
    			<ol> 
    				<li> ThirdListItem  </li>
    				<li>  FourthListItem  </li>
    				<li>  FifthListItem  </li>
    			</ol>
    	</li>
    	<li> SixthListItem  </li>
    </ol>
    
  1. Place the cursor before "sixth item" ^SixthListItem and press backspace

Result: You wont be able to delete "3".
NOTE: Webkit and Opera concatenates fith item with sixth item - FifthListItemSixthListItem. IE puts sixth item in new paragraph. Not sure which behavior is correct.


Scenario 2

  1. The same as above
  2. Put the cursor behind fifth item FifthListItem^ and start pressing delete.

Result: When all text in sixth item gets deleted, number is deleted at the end.

Attachments (2)

8249.patch (945 bytes) - added by Garry Yao 13 years ago.
8249_2.patch (1.8 KB) - added by Garry Yao 13 years ago.

Download all attachments as: .zip

Change History (12)

comment:1 Changed 13 years ago by Jakub Ś

Status: newconfirmed

comment:2 Changed 13 years ago by Satya Minnekanti

Cc: satya_minnekanti@… added

comment:3 Changed 13 years ago by Jakub Ś

Description: modified (diff)

comment:4 Changed 13 years ago by Damian

Keywords: IBM added

Changed 13 years ago by Garry Yao

Attachment: 8249.patch added

comment:5 Changed 13 years ago by Garry Yao

Owner: set to Garry Yao
Status: confirmedreview

Normalize this specific backspace to MS-Word alike behavior which outdent the list item.

comment:6 Changed 13 years ago by Frederico Caldeira Knabben

Status: reviewreview_failed

The idea is good and it works well.

But just to avoid people complaining later, let's have it working well for this case:

  1. Load this HTML and selection:
    <ul>
    	<li>
    		<p>Para 1</p>
    		<p>^Para 2</p>
    	</li>
    </ul>
    
  2. BACKSPACE

After patch, the li will outdent, instead of having the paragraphs merged.

Changed 13 years ago by Garry Yao

Attachment: 8249_2.patch added

comment:7 Changed 13 years ago by Garry Yao

Keywords: Firefox removed
Status: review_failedreview

comment:8 Changed 13 years ago by Frederico Caldeira Knabben

Milestone: CKEditor 3.6.3
Status: reviewreview_passed

comment:9 Changed 13 years ago by Garry Yao

Resolution: fixed
Status: review_passedclosed

Fixed with [7392].

comment:10 Changed 13 years ago by Garry Yao

One more new ticket test failed, post fixed with [7424].

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