Opened 8 years ago

Closed 8 years ago

Last modified 7 years ago

#13728 closed defect (fixed)

[patch][ccla]FF: dijit.Editor - page up/down causes cursor to get stuck when there are no scroll bars

Reported by: Katie Vance Owned by: bill
Priority: high Milestone: 1.7
Component: Editor Version: 1.6.1
Keywords: Cc:
Blocked By: Blocking:

Description (last modified by Katie Vance)

In firefox, when the editor does not have scroll bars, then pressing page up/down will make it so you cannot type in the editor. The editor still has focus, it's just unusable. To get it unstuck, you can press the up and down keys.

Attachments (1)

13728.patch​ (1.9 KB) - added by Katie Vance 8 years ago.
Stop the page up/down events if no scroll bar is visible

Download all attachments as: .zip

Change History (11)

comment:1 Changed 8 years ago by Katie Vance

Description: modified (diff)
Summary: dijit.Editor - page up/down causes cursor to get stuck when there are no scroll bars → FF: dijit.Editor - page up/down causes cursor to get stuck when there are no scroll bars

comment:2 Changed 8 years ago by Douglas Hays

Patch needs a robot testcase.

comment:3 Changed 8 years ago by Katie Vance

attached robot testcase

comment:4 Changed 8 years ago by Douglas Hays

Summary: FF: dijit.Editor - page up/down causes cursor to get stuck when there are no scroll bars → [patch][ccla]FF: dijit.Editor - page up/down causes cursor to get stuck when there are no scroll bars

Changed 8 years ago by Katie Vance

Attachment: 13728.patch​ added

Stop the page up/down events if no scroll bar is visible

comment:5 Changed 8 years ago by Katie Vance

Bill, can you commit this?

comment:6 Changed 8 years ago by Douglas Hays

Owner: changed from Katie Vance to bill

comment:7 Changed 8 years ago by bill

Resolution: → fixed
Status: new → closed

In [26782]:

Avoid editor locking up in firefox after pressing page up / page down, based on patch from Katie, thanks. Fixes #13728 !strict.

comment:8 Changed 8 years ago by bill

Milestone: tbd → 1.7

comment:9 Changed 7 years ago by bill

In [26945]:

Disable test from [26782] for webkit until pagedown bug is fixed, refs #13728, #14193.

comment:10 Changed 7 years ago by bill

In [26946]:

On 1.7 branch, disable test from [26782] for webkit until pagedown bug is fixed, refs #13728, #14193.

Note: See TracTickets for help on using tickets.