Ticket #4736 (closed Bug: fixed)

Opened 3 months ago

Last modified 3 months ago

Page Up/Down keys in wysiwyg not working well

Reported by: garry.yao Owned by: garry.yao
Priority: Normal Milestone: CKEditor 3.1
Component: General Version:
Keywords: Confirmed Review+ Cc:

Description (last modified by garry.yao) (diff)

Environment

WinXP, affected all IE versions, with the following config:

config.docType = '<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">';

Reproducing Procedures

  1. Open a sample page with contents long enough to appear the scrollbar.
  2. Put cursor at the beginning of document and press 'PageDown' key.
  • Actual Result: Though the selection is changed, the scrollbar is not moved a bit at all.
  • Expected Result: The cursor's scrolling into the desired position.

Attachments

4736.patch Download (2.0 KB) - added by garry.yao 3 months ago.
4736_2.patch Download (2.4 KB) - added by garry.yao 3 months ago.

Change History

Changed 3 months ago by garry.yao

  • description modified (diff)

Only occurred when 'config.docType' is set to XHTML.

Changed 3 months ago by garry.yao

Changed 3 months ago by garry.yao

  • keywords Review? added
  • owner set to garry.yao
  • status changed from new to assigned

Changed 3 months ago by fredck

  • keywords Review- added; Review? removed

The idea is good. The only problem is that the key event is being registered everytime the wysiwyg area is created (e.g. switching forth and back to source), which makes the code being executed more than one when hitting the key. The event should be registered just the first time.

Changed 3 months ago by garry.yao

Changed 3 months ago by garry.yao

  • keywords Review? added; Review- removed

Targeting the above mentioned issue.

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 [4588].

Note: See TracTickets for help on using tickets.