Opened 14 years ago

Closed 14 years ago

#4736 closed Bug (fixed)

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)

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 (2)

4736.patch (2.0 KB) - added by Garry Yao 14 years ago.
4736_2.patch (2.4 KB) - added by Garry Yao 14 years ago.

Download all attachments as: .zip

Change History (8)

comment:1 Changed 14 years ago by Garry Yao

Description: modified (diff)

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

Changed 14 years ago by Garry Yao

Attachment: 4736.patch added

comment:2 Changed 14 years ago by Garry Yao

Keywords: Review? added
Owner: set to Garry Yao
Status: newassigned

comment:3 Changed 14 years ago by Frederico Caldeira Knabben

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 14 years ago by Garry Yao

Attachment: 4736_2.patch added

comment:4 Changed 14 years ago by Garry Yao

Keywords: Review? added; Review- removed

Targeting the above mentioned issue.

comment:5 Changed 14 years ago by Frederico Caldeira Knabben

Keywords: Review+ added; Review? removed

comment:6 Changed 14 years ago by Garry Yao

Resolution: fixed
Status: assignedclosed

Fixed with [4588].

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