Ticket #2532815 (closed defect)
Reporter Kreso Kunjas |
Opened: 10/1/12 Last modified: 01/28/13 Status: closed Type: defect Resolution: fixed |
Owner  Derek Gathright |
Target Release: 3.8.1 Priority: P3 (normal) |
|---|---|---|---|
| Summary: | ScrollView paginator doesn't work after using mouse scroll anywhere on page | ||
| Description: | steps to reproduce: if i interact with scrollview, it works, and pagination buttons works again. Tested with opera 12, latest chrome (23) and firefox (15.0.1) |
||
| Type: | defect | Observed in Version: | 3.7.2 |
| Component: | ScrollView | Severity: | S2 (high) |
| Assigned To: | Derek Gathright | Target Release: | 3.8.1 |
| Location: | Library Code | Priority: | P3 (normal) |
| Tags: | Relates To: | ||
| Browsers: | Firefox - Latest,Opera,Chrome | ||
| URL: | http://stage.yuilibrary.com/yui/docs/scrollview/scrollview-paging-example.html | ||
| Test Information: | |||
Change History
|
Posted: 10/1/12
|
|
Posted: 10/1/12
Thanks Kreso. After a quick investigation, it appears that Paginator's _beforeHostScrollTo [1] is referencing the previous interaction's axis (which was a mousewheel on axis 'Y' [2]) and using that as the axis for the API 'next' interaction. I'll investigate further for the appropriate fix. Ideally, scrollview._gesture is completely removed at the end of a gesture cycle, because it can cause issues [3] (such as this) when it lingers around. Also, mousewheel shouldn't have any interaction with the _gesture object, as it isn't a gesture. [1] https://github.com/yui/yui3/blob/v3.7.2/src/scrollview/js/paginator-plugin.js#L226 |
|
Posted: 10/1/12
I'm going to continue more testing, but it appears that simply removing the following lines from paginator-plugin.js resolves the issue - // Set the axis for this event. This was likely required for the 3.5-3.6 releases, but no longer necessary in the 3.7 release which involved a lot of SV cleanup. |
|
Posted: 10/1/12
|
|
Posted: 10/1/12
Fixes #2532815 - MouseWheel no longer makes use of the _gesture property and this can be removed. |
|
Posted: 10/2/12
Checked in to 3.x and will be released in 3.8 |
|
Posted: 10/2/12
|
|
Posted: 10/3/12
Fixes #2532815 - MouseWheel no longer makes use of the _gesture property and this can be removed. |
|
Posted: 11/16/12
Fixes #2532815 - MouseWheel no longer makes use of the _gesture property and this can be removed. |
|
Posted: 12/11/12
Fixes #2532815 - MouseWheel no longer makes use of the _gesture property and this can be removed. |
|
Posted: 12/11/12
Fixes #2532815 - MouseWheel no longer makes use of the _gesture property and this can be removed. |
|
Posted: 12/11/12
Fixes #2532815 - MouseWheel no longer makes use of the _gesture property and this can be removed. |
|
Posted: 12/12/12
|
|
Posted: 12/12/12
|
|
Posted: 01/14/13
Fixes #2532815 - MouseWheel no longer makes use of the _gesture property and this can be removed. |
|
Posted: 01/14/13
Fixes #2532815 - MouseWheel no longer makes use of the _gesture property and this can be removed. |
|
Posted: 01/15/13
Ticket #2532974 was marked as a duplicate of this ticket. |
|
Posted: 01/28/13
Moving all checked in bugs to 3.8.1. |
|
Posted: 01/28/13
|
i have done some testing. if i override _mousewhell private function to empty function the problem is fixed.
i also observed that after mousewhell event fires and _mousewhell function gets executed pagination stops working.
after that if i call paginate.next() or prev function the scrollview scrollTo function recieves wrong parameters (null instead of passed in value). maybe that should help..