Ticket #4673 (closed Bug: fixed)

Opened 4 months ago

Last modified 3 months ago

Undo not available straight away if shift key is used to enter first character.

Reported by: JoeK Owned by: garry.yao
Priority: Normal Milestone: CKEditor 3.1
Component: Core : Undo/Redo Version: SVN (CKEditor)
Keywords: IBM Confirmed Review+ Cc: dchojna@…

Description

When entering some text into the editor use the shift key to capitalize the first character. Observe that the Undo button is not active until about 15 characters have been entered.

Attachments

4673.patch Download (2.7 KB) - added by garry.yao 4 months ago.
4673_2.patch Download (2.7 KB) - added by garry.yao 3 months ago.
4673_3.patch Download (1.2 KB) - added by garry.yao 3 months ago.

Change History

Changed 4 months ago by garry.yao

  • keywords Confirmed added
  • owner set to garry.yao
  • version changed from 3.0.1 to SVN (CKEditor)
  • status changed from new to assigned

Nice catch, this's something that definitely need to fix right now.

Changed 4 months ago by garry.yao

Changed 4 months ago by garry.yao

Update key map of editing keys, modifier keys and navigation keys according to:  keyboard keys

Changed 4 months ago by garry.yao

  • keywords Review? added

Changed 4 months ago by fredck

  • keywords Review- added; Review? removed

Please remove the space and tab keys from the editing list as having snapshots for each space reduces the undo stack too much, considering that it's a very very common key (the most common).

Changed 3 months ago by garry.yao

Changed 3 months ago by garry.yao

  • keywords Review? added; Review- removed

Changed 3 months ago by fredck

  • keywords Review+ added; Review? removed

Changed 3 months ago by garry.yao

  • status changed from assigned to closed
  • resolution set to fixed

Fixed with [4607].

Changed 3 months ago by damo

  • status changed from closed to reopened
  • resolution fixed deleted

This is still reproducible, although the steps to reproduce are slightly different.

1. Use Ajax sample.
2. Create editor and focus into body.
3. Press CAPS Lock once. (This means toggle once, it does not matter if it is on or off)
4. Type into the editor.

Notice that undo does not activate.

Reproduced on FF 3.5 & IE 7

Changed 3 months ago by garry.yao

Changed 3 months ago by garry.yao

  • keywords Review? added; Review+ removed

Changed 3 months ago by fredck

  • keywords Review+ added; Review? removed

Changed 3 months ago by garry.yao

  • status changed from reopened to closed
  • resolution set to fixed

Fixed with [4817].

Note: See TracTickets for help on using tickets.