Closed
Bug 728135
Opened 14 years ago
Closed 14 years ago
should use clamped scroll position in nsGfxScrollFrameInner::ScrollTo instead of unclamped one
Categories
(Core :: Layout, defect)
Core
Layout
Tracking
()
RESOLVED
INVALID
People
(Reporter: tnikkel, Unassigned)
References
Details
Attachments
(1 file)
|
1004 bytes,
patch
|
mstange
:
feedback-
|
Details | Diff | Splinter Review |
No description provided.
|
Reporter | |
Comment 1•14 years ago
|
||
Bug 627651 replaced mDestination used in the same place with aScrollPosition. Was that intended? Seems like we want the clamped scroll position here.
Attachment #598162 -
Flags: review?(mstange)
|
||
Comment 2•14 years ago
|
||
Comment on attachment 598162 [details] [diff] [review]
patch
Yes, this was intended. I think being near a wall shouldn't have any influence on the animation. Using the clamped position would make the scroll motion slower (in terms of velocity, not completion time).
To me, the previous behavior felt like the edges of the scrolled area had some kind of force field that protected them from a fast scroll collision. It didn't feel right.
Attachment #598162 -
Flags: review?(mstange) → feedback-
|
|
Reporter | |
Comment 3•14 years ago
|
||
Alright.
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → INVALID
|
|
Reporter | |
Updated•14 years ago
|
Assignee: tnikkel → nobody
You need to log in
before you can comment on or make changes to this bug.
Description
•