Opened 14 years ago
Closed 14 years ago
#4795 closed Bug (fixed)
[IE] Deleting focused horizontal line/table by DEL key generates javascript error
| Reported by: | pomu0325 | Owned by: | Garry Yao |
|---|---|---|---|
| Priority: | Normal | Milestone: | CKEditor 3.2 |
| Component: | General | Version: | 3.0.1 |
| Keywords: | Confirmed IE Review+ | Cc: | pomu@… |
Description
To Reproduce
- Open sample page with IE
- Click HR button from toolbar
- Press 'Delete' key
- Error occurs on line.210 of selection/plugin.js
Attachments (2)
Change History (9)
comment:1 Changed 14 years ago by
| Summary: | [IE] Deleting horizontal line generates javascript error → [IE] Deleting horizontal line/table generates javascript error |
|---|
comment:2 follow-up: 3 Changed 14 years ago by
| Keywords: | Confirmed IE added |
|---|---|
| Milestone: | → CKEditor 3.2 |
Confirmed with IE8. (The HR must be selected first)
comment:3 Changed 14 years ago by
| Summary: | [IE] Deleting horizontal line/table generates javascript error → [IE] Deleting focused horizontal line/table by DEL key generates javascript error |
|---|
Replying to fredck:
Confirmed with IE8. (The HR must be selected first)
Sorry, select step was missing.
comment:4 Changed 14 years ago by
I found out that IE8 hide focus after removing "Control" type object (such as table, horizontal line). So I wrote a workaround to move caret to the next element.
Changed 14 years ago by
| Attachment: | 4795_2.patch added |
|---|
comment:5 Changed 14 years ago by
| Keywords: | Review? added |
|---|---|
| Owner: | set to Garry Yao |
| Status: | new → assigned |
Providing a simplified patch inspired by pomu's idea.
comment:6 Changed 14 years ago by
| Keywords: | Review+ added; Review? removed |
|---|
Note: See
TracTickets for help on using
tickets.
Deleting focused table by 'Delele' key also generates same error.