Bug 24477 – REGRESSION (r41467): Page Down key scrolls two pages

• Component: WebKit Misc.
• Status: RESOLVED FIXED

[nobody]

hitting "Page Down" keyboard button broken (scrolls more than one "page", i.e. more than one screen distance)

problem observed in r41521, not sure if that is first appearance of problem, but this bug appeared very recently.

[Alexey Proskuryakov]

Are you seeing this on all sites? I cannot reproduce the problem with r41550 (although I have a notebook, so it's Fn+Down, not PgDown for me).

[nobody]

Yes, every website.

The "Page Down" key is located on keyboard among Home/End/Delete/Page Up/Page Down/Insert buttons.

(btw, "Page Up" scrolls the wrong distance, also.)

Problem continues in r41545.

[Alexey Proskuryakov]

Weird, it suddenly started to happen to me. Maybe I was testing an old version previously.

Logging editing commands, I see that doCommandBySelector:"scrollPageDown:" is invoked twice for each PageDown.

[Rajiv Aaron Manglani]

i'm seeing this in WebKit r41545 with safari 4 public beta installed: Version 4 Public Beta (5528.16, r41545), on a mac with an "extended keyboard" with actual page up/page down keys.

the problem is not just with page down but also with page up.

[Alexey Proskuryakov]

<rdar://problem/6674184>

[Rajiv Aaron Manglani]

this was broken in Changeset 41467 for trunk/WebKit/mac/WebView/WebHTMLView.mm in order to fix bug #24079. see http://trac.webkit.org/changeset/41467

building webkit r41660 with that change reversed fixes the problems described in this bug.

[Rajiv Aaron Manglani]

Created attachment 28572 [details]
patch to fix double page down/page up bug

since the selector is passed to super, consider the event handled.

code originally introduced in http://trac.webkit.org/changeset/39549

[Adele Peterson]

Comment on attachment 28572 [details]
patch to fix double page down/page up bug

This will cause https://bugs.webkit.org/show_bug.cgi?id=23060 to show up again.  To reproduce that bug, try running Mail.app with your patched WebKit.  Focus a message in the preview pane, and you'll see that Home, End, Page Up, and Page Down don't work.

[Adele Peterson]

Here's an experiment we should try.  If we added the selectors for Page Up/Down and Home/End to the Editor command table in WebCore, would we still need eventHandled to be false here to get the Mail testcase to work?

[Darin Adler]

I値l take this one.

[Darin Adler]

http://trac.webkit.org/changeset/41793

[Mark Rowe (bdash)]

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

[nobody]

This problem is not fixed.

Some recent versions seemed OK.

r41760 again scrolls too far. 

It seems to be not quite a full page extra, but it is scrolling the wrong distance.

[Darin Adler]

The difference in distance scrolled is probably due to the fact that we now use WebCore's scrolling code instead of WebFrameView's.

The old bug was absolutely critical, the new bug simply makes the Mac version match the Windows version. While I want to fix the new bug, it's *not* the same thing, so reopening this bug report isn't quite right.

[Doug Harris]

(In reply to comment #13)
> This problem is not fixed.
> 
> Some recent versions seemed OK.
> 
> r41760 again scrolls too far. 

According to comment #12, this was committed in r41793 -- which is after r41760.

There has been no new nightly build since r41760 (I don't know why).

[Darin Adler]

Please try again with a version newer than r41793.

[nobody]

seems to be OK in r41844.

Thanks!

[Cameron Zwarich (cpst)]

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