Opened 15 years ago

Closed 15 years ago

#3212 closed Bug (fixed)

CKReleaser is breaking hotkeys in CKEditor

Reported by: Martin Kou Owned by: Wiktor Walc
Priority: Must have (possibly next milestone) Milestone: CKEditor 3.0
Component: General Version: SVN (CKEditor) - OLD
Keywords: Oracle Review+ Cc: Senthil

Description

To reproduce:

  1. Add Tab_Issue.2.html to the _samples directory of SVN trunk.
  2. Compress the trunk with CKReleaser.
  3. Open Tab_Issue.2.html within the release package.
  4. Press Tab and Shift-Tab from inside the editing area, you'll see different buggy behaviors described in #3132.
  5. Press Ctrl-B in the editing area. You'll see other buggy behaviors depending on the browser.

Attachments (1)

3212.patch (598 bytes) - added by Wiktor Walc 15 years ago.

Download all attachments as: .zip

Change History (5)

comment:1 Changed 15 years ago by Wiktor Walc

Owner: set to Wiktor Walc

Changed 15 years ago by Wiktor Walc

Attachment: 3212.patch added

comment:2 Changed 15 years ago by Wiktor Walc

Keywords: Review? added

The problem was in invalid redeclaration of the cancel variable.

As a result CKPackager renamed the cancel variable to another letter and used that letter in

if ( cancel )
	event.preventDefault( true );

which caused invalid results.

There is no need to redeclare this variable.

comment:3 Changed 15 years ago by Frederico Caldeira Knabben

Keywords: Review+ added; Review? removed

Oppss... it looks like we need a jslint run here.

comment:4 Changed 15 years ago by Wiktor Walc

Resolution: fixed
Status: newclosed

Fixed with [3291].

Note: See TracTickets for help on using tickets.
© 2003 – 2022, CKSource sp. z o.o. sp.k. All rights reserved. | Terms of use | Privacy policy