725700 - Keyboard arrow keys and scrollbar buttons should have consistent scrolling distances

Status: VERIFIED FIXED

(Core :: General, defect)

Description

[Jared Wein [:jaws] (please needinfo? me)]

Bug 635645 made keyboard arrow keys scroll two lines at a time. Bug 710373 made the scrollbar buttons scroll three lines at a time.

These scrolling distances should be consistent, and we should make the arrow keys scroll three lines at a time instead of two.

To make sure that these distances stay consistent going forward, these distances should use a shared preferences (possibly updated through about:config).

Comment 1

[Jared Wein [:jaws] (please needinfo? me)]

Attachment: Patch for bug

Comment 2

[Mozilla RelEng Bot]

Autoland Patchset:
	Patches: 597236
	Branch: mozilla-central => try
	Destination: http://hg.mozilla.org/try/pushloghtml?changeset=6c47f76858c8
Try run started, revision 6c47f76858c8. To cancel or monitor the job, see: https://tbpl.mozilla.org/?tree=Try&rev=6c47f76858c8

Comment 3

[Jared Wein [:jaws] (please needinfo? me)]

Attachment: Patch for bug v2

The push to try showed that docshell/test/navigation/test_bug430723.html was broken since the test reached the bottom of the window before the scroll iterations were finished and the scroll event wouldn't fire anymore on DOWN key.

I've updated the test to scroll down less since the distance of scrolling has been increased.

Comment 4

[Mozilla RelEng Bot]

Try run for 6c47f76858c8 is complete.
Detailed breakdown of the results available here:
    https://tbpl.mozilla.org/?tree=Try&rev=6c47f76858c8
Results (out of 212 total builds):
    success: 168
    warnings: 30
    failure: 14
Builds (or logs if builds failed) available at:
http://ftp.mozilla.org/pub/mozilla.org/firefox/try-builds/[e-mail]-6c47f76858c8

Comment 5

[Mozilla RelEng Bot]

Autoland Patchset:
	Patches: 597601
	Branch: mozilla-central => try
	Destination: http://hg.mozilla.org/try/pushloghtml?changeset=c6f9bd404ae0
Try run started, revision c6f9bd404ae0. To cancel or monitor the job, see: https://tbpl.mozilla.org/?tree=Try&rev=c6f9bd404ae0

Comment 6

[Mozilla RelEng Bot]

Try run for c6f9bd404ae0 is complete.
Detailed breakdown of the results available here:
    https://tbpl.mozilla.org/?tree=Try&rev=c6f9bd404ae0
Results (out of 209 total builds):
    success: 145
    warnings: 13
    failure: 51
Builds (or logs if builds failed) available at:
http://ftp.mozilla.org/pub/mozilla.org/firefox/try-builds/[e-mail]-c6f9bd404ae0

Comment 7

[Jared Wein [:jaws] (please needinfo? me)]

review ping? only change from previous r+'d version is the test update.

Comment 8

[Jared Wein [:jaws] (please needinfo? me)]

https://hg.mozilla.org/integration/fx-team/rev/74cb4c127dc1

Comment 9

[Rob Campbell [:rc] (:robcee)]

https://hg.mozilla.org/mozilla-central/rev/74cb4c127dc1

Comment 10

[Mihaela Velimiroviciu (:mihaelav)]

Mozilla/5.0 (Windows NT 6.1; rv:13.0) Gecko/20100101 Firefox/13.0
Mozilla/5.0 (X11; Linux i686; rv:13.0) Gecko/20100101 Firefox/13.0
Mozilla/5.0 (Macintosh; Intel Mac OS X 10.6; rv:13.0) Gecko/20100101 Firefox/13.0

Verified that both keyboard arrows and scrollbar buttons scroll about 3 lines at once.