251351 - No event point for wheel scroll event

Status: RESOLVED FIXED

(Core Graveyard :: GFX: BeOS, defect)

Description

[Sergei Dolgov]

Current implementation in widget/src/beos/nsWindow.cpp reports fixed point
(100,100) for NS_MOUSE_SCROLL.
It results in stupid bugs on pages with scrollable iframe, like stopping of
scrolling main content when iframe gets visible. On some pages it is impossible
at all to scroll main content.

Fix seems simple - adding GetMouse() to get cursor position,
but it leads to strange behaviour - if you reached bottom of page, scroll-up
moves to top of page, instead moving up for scroll-step.

So investigating

Comment 1

[Sergei Dolgov]

oops, no problem with scrol-up. was stupid side-effect of another change in code.

Submitting fix soon

Comment 2

[Sergei Dolgov]

Attachment: patch. diff -up4

gets cursor coordinates and assigns those to even point.
compare wheel behaviour before and after patch 
here:
http://beos.spb.ru/mozilla/frametest/testmain.html

before patch wheel scrolling scrolls rather iframe than main page.
after patch wheel scroll object at which it's over.

Comment 3

[Sergei Dolgov]

Comment on attachment 153224 [details] [diff] [review]
patch. diff -up4

need to remove XXX comment

Comment 4

[Sergei Dolgov]

Attachment: patch

same as above but comment removed.

Comment 5

[Sergei Dolgov]

Comment on attachment 153227 [details] [diff] [review]
patch

asking for review

Comment 6

[tqh]

Comment on attachment 153227 [details] [diff] [review]
patch

r=[e-mail]

Comment 7

[Sergei Dolgov]

fixed.
new revision: 1.82

Comment 8

[Niels Reedijk]

Checkin on AVIARY by db48x%yahoo.com 
 
"Check in several BeOS bugs that are already checked in on the trunk. All are 
r=twh ([e-mail]) except the last which is r=Sergei Dolgov 
([e-mail]). a=carte blanche ([03:53:40] 
<nielx!~nielx@84.119.151.125> We've been basically given a carte blanche for 
any change that involves the beos-only code)"