Opened 14 years ago
Closed 12 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)
Change History (21)
Changed 14 years ago by
| Attachment: | ckeditor_empty_line.png added |
|---|
comment:1 Changed 14 years ago by
| Status: | new → confirmed |
|---|
comment:2 Changed 14 years ago by
| Cc: | satya_minnekanti@… added |
|---|
comment:3 Changed 14 years ago by
comment:4 Changed 14 years ago by
| Version: | 3.0 → 3.4.1 |
|---|
comment:5 Changed 14 years ago by
| Version: | 3.4.1 → 3.0 |
|---|
comment:6 Changed 14 years ago by
| 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> </p>
<table border="1" cellpadding="1" cellspacing="1" style="width: 500px; ">
<tbody>
<tr><td> </td></tr>
</tbody>
</table>
<p> </p>
comment:7 Changed 13 years ago by
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.
Changed 13 years ago by
| Attachment: | 6217.patch added |
|---|
comment:9 Changed 13 years ago by
| Owner: | set to Garry Yao |
|---|---|
| Status: | confirmed → review |
comment:10 Changed 13 years ago by
| Milestone: | → CKEditor 3.6.4 |
|---|
The proposed patch fixes native keystroke limitation regard del/backspace key at the boundary of table.
comment:11 Changed 13 years ago by
| Status: | review → 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 13 years ago by
| Attachment: | 6217_2.patch added |
|---|
comment:12 Changed 13 years ago by
| Component: | General → Core : Keystrokes |
|---|---|
| Status: | review_failed → review |
comment:13 follow-up: 15 Changed 13 years ago by
comment:15 Changed 12 years ago by
| Status: | review → review_failed |
|---|
comment:17 Changed 12 years ago by
| Status: | review → review_passed |
|---|
comment:18 Changed 12 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | review_passed → closed |
Fixed with [7537].
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.