Closed
Bug 251351
Opened 21 years ago
Closed 21 years ago
No event point for wheel scroll event
Categories
(Core Graveyard :: GFX: BeOS, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: sergei_d, Assigned: sergei_d)
Details
(Keywords: fixed-aviary1.0)
Attachments
(1 file, 1 obsolete file)
1.25 KB,
patch
|
thesuckiestemail
:
review+
|
Details | Diff | Splinter Review |
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
Assignee | ||
Comment 1•21 years ago
|
||
oops, no problem with scrol-up. was stupid side-effect of another change in code.
Submitting fix soon
Assignee: beos → sergei_d
Assignee | ||
Comment 2•21 years ago
|
||
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.
Attachment #153224 -
Flags: review+
Assignee | ||
Comment 3•21 years ago
|
||
Comment on attachment 153224 [details] [diff] [review]
patch. diff -up4
need to remove XXX comment
Attachment #153224 -
Attachment is obsolete: true
Assignee | ||
Comment 4•21 years ago
|
||
same as above but comment removed.
Assignee | ||
Comment 5•21 years ago
|
||
Comment on attachment 153227 [details] [diff] [review]
patch
asking for review
Attachment #153227 -
Flags: review?(thesuckiestemail)
Attachment #153227 -
Flags: review?(thesuckiestemail) → review+
Assignee | ||
Comment 7•21 years ago
|
||
fixed.
new revision: 1.82
Status: NEW → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED
Comment 8•20 years ago
|
||
Checkin on AVIARY by db48x%yahoo.com
"Check in several BeOS bugs that are already checked in on the trunk. All are
r=twh (thesuckiestemail@yahoo.se) except the last which is r=Sergei Dolgov
(sergei_d@fi.tartu.ee). 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)"
Keywords: fixed-aviary1.0
Updated•16 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•