build id: 1999052619 platform: windwos nt to reproduce, go to the above url or to any other page that's too wide for the default apprunner window to display without drawing scrollbars. when the page has loaded, use the up and down arrow keys to scroll up and down, and then try the left and right arrow keys to scroll from left to right. result: you can't. expected result: you can.

Exhibited upon Lose98, as well.

Verified the bug on the following builds: - 1999071417 Viewer on Windows NT 4.0 SP4 - 1999071417 Apprunner on Windows NT 4.0 SP4 - M7 Apprunner on Windows NT 4.0 SP4 Verified the URL as a valid test case - actually, the bug can be reproduced on virtually any page (including a Bugzilla page) simply by narrowing the browser window enough. Also added the [4.xP] identifier.

Is there a Target Milestone for this bug? It's a reasonably important feature (and I wouldn't think it's too hard to implement, considering vertical arrows already work).

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

This is still a problem as of Build ID 1999112108 (on Linux)

Marking all as it doesn't work on Linux or NT.

Bulk moving [testcase] code to new testcase keyword. Sorry for the spam!

Setting the keyword all open [4.xp] bugs to 4xp.

I'm torn on whether this should be PDT+ or not. But I don't have time to do it tonight. I'll move it to M15 and people can argue over it there if they choose.

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

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

jan, i'll take this one off your hands. lemme know if you want it back. ;-)

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

Mass-moving bugs out of M15 that I won't get to. Will refit individual milestones after moving them.

Nominating for beta2. This is an important feature for wide pages, and works in 4.x. I wouldn't think it should be too hard to fix...

The problems happens on Sun's platform, too, on pre-beta as well as a local cvs-child build(2000030708). This is serious regression from 4.7, so set the priority to critical.

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

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

i un-dup'ed 35558 again for now, but it is the same problem. The submitter has attached a patch for this problem, however. Someone should take a look.

The patch I posted to fix bug 38558 is almost same as patch from Matt Bissiri Horizontal scrolling is working for me now, can someone review this patches ?

Adding myself to CC list

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

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

I updated the patch from Matt Bissiri (Feb 18) to use |nsCAutoString(...).EqualsWithConversion(...)|, and built and tested on linux. Works for me :-).

Okay, fix checked in. Thanks for the patches.

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

Is anybody else seeing that one can horizontally scroll to the left with the keys and then cycle around? I guess a new bug should be opened...

david, which platform? i don't get the cycling around on linux (browser window)...

Linux. I was using my debug tree, which has some other changes in it, although I don't think they would affect this. Are you lookin at a page that scrolls horizontally. (You may need to go all the way to the right first, because it may need to be at an uneven offset -- I forget.)

i was looking at a page whose content is wider than the browser's window. here are the steps i took (lemme know if you ran into the problem a different way): 1. a bugzilla report page such as this one. 2. i make the browser window small enough so that the horizontal scrollbar appears. 3. hit the right arrow key so that it scrolls all the way to the right. 4. hit the left arrow key so that it scrolls all the way to the left. 5. keep hitting the left arrow key. result: it doesn't cycle. admittedly, i'm using an optimized [commercial] build from this am [2000.05.16.08]...

dbaron : I don't see this 'wrap-around' with today's verification builds mac/win32/linux, or with a debug linux build from this morning. However, I note that Matt Bissiri had this in the patch " // XXX Should we be paranoid and check for out of bounds x here, // even though ScrollingView::ScrollTo clamps x and y values?". Matt -- comments? (On an related note, you need to set focus to the content area before you can scroll, but that's a separate bug (the number of which escapes me at the moment)).

I can't see it anymore either...

woo! verif using 2000.05.18.08 on winNT, mac and linux.