Ticket #6217 (closed Bug: fixed)

Opened 4 years ago

Last modified 2 years ago

Cannot delete a line (paragraph) above the table

Reported by: wwalc Owned by: garry.yao
Priority: Normal Milestone: CKEditor 3.6.4
Component: Core : Keystrokes Version: 3.0
Keywords: IBM Cc: satya_minnekanti@…, monahant@…

Description

If you create an empty paragraph and then a table, then you are not able to remove that paragraph (above) anymore (see the screenshot).

Confirmed in Safari 4 and IE7, IE8.

Attachments

ckeditor_empty_line.png (14.7 KB) - added by wwalc 4 years ago.
6217.patch (3.0 KB) - added by garry.yao 2 years ago.
6217_2.patch (5.6 KB) - added by garry.yao 2 years ago.

Change History

Changed 4 years ago by wwalc

comment:1 Changed 4 years ago by wwalc

  • Status changed from new to confirmed

comment:2 Changed 4 years ago by satya

  • Cc satya_minnekanti@… added

comment:3 Changed 4 years ago by dinu

This is one of the worst bugs, I too would want to see it fixed. Note: in FF it's possible to remove the empty paragraph by quickly pressing del and arrow down. It should be an easy logic to implement, when the user presses del or backspace in an empty block the block should be removed and the cursor positioned in the next or previous existing block.

comment:4 Changed 4 years ago by dinu

  • Version changed from 3.0 to 3.4.1

comment:5 Changed 4 years ago by alfonsoml

  • Version changed from 3.4.1 to 3.0

comment:6 Changed 3 years ago by j.swiderski

  • Keywords IBM added

This issue was duplicated in #8108.

As stated by @satya it was fixed for firefox in #7289

Currently this issue is reproducible in all IEs and Safari.

In Chrome if you place the cursor in first paragraph and try to remove it using delete, than the paragraph and table will be selected and removed. Removing the paragraph with backspace is impossible.

Sample source code taken from #8108:

<p>&nbsp;</p>
  <table border="1" cellpadding="1" cellspacing="1" style="width: 500px; ">
     <tbody>
      <tr><td>&nbsp;</td></tr>
     </tbody>
  </table>
<p>&nbsp;</p>

comment:7 Changed 3 years ago by j.swiderski

It is impossible to remove table in all IE's and Webkit browsers. Opera and Firefox work fine.

Open any sample, press CRTL+A, CRTL+X, Enter and insert table. Now try to remove the line above the table with backspace or delete.

NOTE: In Webkit pressing delete selects whole paragraph and table. Next delete press will remove the table.

comment:8 Changed 3 years ago by j.swiderski

#8606 has been marked as duplicate.

Changed 2 years ago by garry.yao

comment:9 Changed 2 years ago by garry.yao

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

comment:10 Changed 2 years ago by garry.yao

  • Milestone set to CKEditor 3.6.4

The proposed patch fixes native keystroke limitation regard del/backspace key at the boundary of table.

comment:11 Changed 2 years ago by fredck

  • Status changed from review to review_failed

The idea is good. We'll at least align its behavior to FF.

The patch is not working with IE9 (BACKSPACE after) and IE9+Compat (BACKSPACE after and DEL before).

Changed 2 years ago by garry.yao

comment:12 Changed 2 years ago by garry.yao

  • Status changed from review_failed to review
  • Component changed from General to Core : Keystrokes

comment:13 follow-up: ↓ 15 Changed 2 years ago by garry.yao

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

comment:14 Changed 2 years ago by tmonahan

  • Cc monahant@… added

Adding to CC list

comment:15 in reply to: ↑ 13 Changed 2 years ago by fredck

  • Status changed from review to review_failed

Replying to garry.yao:

http://ckeditor.t/dt/core/dom/range.html

This one is failing.

comment:16 Changed 2 years ago by garry.yao

  • Status changed from review_failed to review

Test is fixed.

comment:17 Changed 2 years ago by fredck

  • Status changed from review to review_passed

comment:18 Changed 2 years ago by garry.yao

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

Fixed with [7537].

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