Opened 7 years ago

Closed 5 years ago

#6217 closed Bug (fixed)

Cannot delete a line (paragraph) above the table

Reported by: Wiktor Walc 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 (3)

ckeditor_empty_line.png (14.7 KB) - added by Wiktor Walc 7 years ago.
6217.patch (3.0 KB) - added by Garry Yao 5 years ago.
6217_2.patch (5.6 KB) - added by Garry Yao 5 years ago.

Download all attachments as: .zip

Change History (21)

Changed 7 years ago by Wiktor Walc

Attachment: ckeditor_empty_line.png added

comment:1 Changed 7 years ago by Wiktor Walc

Status: newconfirmed

comment:2 Changed 7 years ago by Satya Minnekanti

Cc: satya_minnekanti@… added

comment:3 Changed 7 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 7 years ago by Dinu

Version: 3.03.4.1

comment:5 Changed 7 years ago by Alfonso Martínez de Lizarrondo

Version: 3.4.13.0

comment:6 Changed 6 years ago by Jakub Ś

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 6 years ago by Jakub Ś

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 6 years ago by Jakub Ś

#8606 has been marked as duplicate.

Changed 5 years ago by Garry Yao

Attachment: 6217.patch added

comment:9 Changed 5 years ago by Garry Yao

Owner: set to Garry Yao
Status: confirmedreview

comment:10 Changed 5 years ago by Garry Yao

Milestone: CKEditor 3.6.4

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

comment:11 Changed 5 years ago by Frederico Caldeira Knabben

Status: reviewreview_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 5 years ago by Garry Yao

Attachment: 6217_2.patch added

comment:12 Changed 5 years ago by Garry Yao

Component: GeneralCore : Keystrokes
Status: review_failedreview

comment:13 Changed 5 years ago by Garry Yao

Last edited 5 years ago by Frederico Caldeira Knabben (previous) (diff)

comment:14 Changed 5 years ago by Teresa Monahan

Cc: monahant@… added

Adding to CC list

comment:15 in reply to:  13 Changed 5 years ago by Frederico Caldeira Knabben

Status: reviewreview_failed

Replying to garry.yao:

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

This one is failing.

comment:16 Changed 5 years ago by Garry Yao

Status: review_failedreview

Test is fixed.

comment:17 Changed 5 years ago by Frederico Caldeira Knabben

Status: reviewreview_passed

comment:18 Changed 5 years ago by Garry Yao

Resolution: fixed
Status: review_passedclosed

Fixed with [7537].

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