Ticket #8249 (closed Bug: fixed)

Opened 3 years ago

Last modified 2 years ago

[FF]: Problems with deleting last list item

Reported by: j.swiderski 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 j.swiderski) (diff)

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

8249.patch (945 bytes) - added by garry.yao 3 years ago.
8249_2.patch (1.8 KB) - added by garry.yao 3 years ago.

Change History

comment:1 Changed 3 years ago by j.swiderski

  • Status changed from new to confirmed

comment:2 Changed 3 years ago by satya

  • Cc satya_minnekanti@… added

comment:3 Changed 3 years ago by j.swiderski

  • Description modified (diff)

comment:4 Changed 3 years ago by damo

  • Keywords IBM added

Changed 3 years ago by garry.yao

comment:5 Changed 3 years ago by garry.yao

  • Owner set to garry.yao
  • Status changed from confirmed to review

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

comment:6 Changed 3 years ago by fredck

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

comment:7 Changed 3 years ago by garry.yao

  • Status changed from review_failed to review
  • Keywords Firefox removed

comment:8 Changed 3 years ago by fredck

  • Status changed from review to review_passed
  • Milestone set to CKEditor 3.6.3

comment:9 Changed 3 years ago by garry.yao

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

Fixed with [7392].

comment:10 Changed 2 years ago by garry.yao

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

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