3999 - Scrolling in a form, text area or option list, scrolls entire page

Status: VERIFIED FIXED

(Core :: DOM: UI Events & Focus Handling, defect, P1)

Description

[Ariel Shkedi]

If you use the keyboard to scroll a textarea or an option list, the entire web
page scrolls along with it.

I wasn't able to reproduce this, but in the samples 9 page, the frame with no
scroll bars, I was able to make it scroll by selecting the text and moving
the mouse down (this is normal for pages with scroll bars), I was not able to
make it scroll with the keyboard, but I was with the wheel on my mouse.

Comment 1

[rickg]

Tom: This looks like an event problem. Also note that RodS made a change
recently to add support for mouse wheel. That may be a factor, so call him.

Comment 2

[joki (gone)]

No, its all just event stuff.  Basically the current scrolling code needs to
move elsewhere but I haven't gotten around to it yet.  Since it currently lives
in the actual scrollframe code it scrolls whenever an event passes through a
scrollframe, even if that event is destined for text fields or some such.
Might need to add an api or two to be able to do the same thing outside of
scollframe.  On my list.

Comment 3

[joki (gone)]

*** Bug 4973 has been marked as a duplicate of this bug. ***

Comment 4

[cpratt]

*** Bug 4979 has been marked as a duplicate of this bug. ***

Comment 5

[basic]

*** Bug 5167 has been marked as a duplicate of this bug. ***

Comment 6

[Eli Goldberg]

*** Bug 5350 has been marked as a duplicate of this bug. ***

Comment 7

[joki (gone)]

*** Bug 5360 has been marked as a duplicate of this bug. ***

Comment 8

[vidur (gone)]

Moving off M5 radar since Tom is out of town.

Comment 9

[joki (gone)]

Moving back to M5.  I have a fix for this in my tree.

Comment 10

[joki (gone)]

Okay, this should be good now.  Also fixed editor behavior by specifcially
allowing pageup/down and arrow keys but cancelling spaces.

Comment 11

[Simon Fraser [no longer active]]

Reopening. Up and down arrow keys continue to scroll the editor window content,
instead of moving the caret up and down (which isn't implemented yet, but soon
will be).

Comment 12

[chris hofmann]

so does this become m6?

Comment 13

[Simon Fraser [no longer active]]

Yes. Current state is not so bad that it interferes with testing. Set to M6.

Comment 14

[joki (gone)]

The behavior, however, is still fixed.  Because expanding selection down using
the arrow keys isn't working right now the editor is allowing the key through,
therefore scrolling.  If the editor, or just the selection code, chooses to use
and then consume the arrow keys, then scrolling will not occur.

If you wish to prevent the scrolling the change the editor code to consume the
arrow key.  Re-marking fixed.

Comment 15

[Jan Carpenter]

appears to be fixed.
Verified build: 1999-06-02-08-M7

Comment 16

[Doron Rosenberg (IBM)]

This seems to have popped up again with today's m16 builds.

See bug 35952. Space bar causes page to scroll.