Ticket #4795 (closed Bug: fixed)

Opened 4 years ago

Last modified 4 years ago

[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

4795.patch (1.7 KB) - added by pomu0325 4 years ago.
workaroung patch for IE8Compat mode
4795_2.patch (1.8 KB) - added by garry.yao 4 years ago.

Change History

comment:1 Changed 4 years ago by pomu0325

  • Summary changed from [IE] Deleting horizontal line generates javascript error to [IE] Deleting horizontal line/table generates javascript error

Deleting focused table by 'Delele' key also generates same error.

comment:2 follow-up: ↓ 3 Changed 4 years ago by fredck

  • Keywords Confirmed IE added
  • Milestone set to CKEditor 3.2

Confirmed with IE8. (The HR must be selected first)

comment:3 in reply to: ↑ 2 Changed 4 years ago by pomu0325

  • Summary changed from [IE] Deleting horizontal line/table generates javascript error to [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 4 years ago by pomu0325

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 4 years ago by pomu0325

workaroung patch for IE8Compat mode

Changed 4 years ago by garry.yao

comment:5 Changed 4 years ago by garry.yao

  • Owner set to garry.yao
  • Status changed from new to assigned
  • Keywords Review? added

Providing a simplified patch inspired by pomu's idea.

comment:6 Changed 4 years ago by fredck

  • Keywords Review+ added; Review? removed

comment:7 Changed 4 years ago by garry.yao

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

Fixed with [5010].

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