Ticket #2531486 (closed defect)

Reporter


Derek Gathright
Opened: 11/23/11
Last modified: 07/21/12
Status: closed
Type: defect
Resolution: fixed

Owner


Derek Gathright
Target Release: 3.6.0
Priority: P3 (normal)
Summary: ScrollView bounds twice
Description:

Message



I'd like to fix the probrem of ScrollView in 3.4.1.

The content bounds twice when I flick it from the edge beyond the end.
Particuraly this behavior can be observed when both the parameters "bounce" and "deceleration" are set near "1".

It seems that the cause of this probrem is that the handler of gesturemoveend event conflicts with that of flick event.
However the position of the content is set to this edge in _onGestureMoveEnd(),
it is set continually in _flickFrame() until "velocity" is over the threshold(=0.015).

I tried to fix this problem like following statement, for your information.



$ diff -u scrollview-base{,.new}.js
--- scrollview-base.js 2011-11-21 11:10:54.000000000 +0900

+++ scrollview-base.new.js 2011-11-21 17:25:23.000000000 +0900

@@ -740,6 +740,12 @@
vel = sv._currentVelocity,
step = ScrollView.FRAME_STEP;

+ if (sv._snapToEdge) {

+ sv._flicking = false;

+ sv._killTimer(false);

+ return;

+ }

+
if (scrollsVertical) {
maxY = sv._maxScrollY;
minY = sv._minScrollY;


Thanks,
Yuta Igarashi yuuigara@yahoo-corp.jp
Yahoo! JAPAN

Type: defect Observed in Version: 3.4.1
Component: ScrollView Severity: S3 (normal)
Assigned To: Derek Gathright Target Release: 3.6.0
Location: Library Code Priority: P3 (normal)
Tags: Relates To:
Browsers: N/A
URL:
Test Information:

Change History

Derek Gathright

YUI Developer

Posted: 12/14/11
  • sprint changed from sprint 1 to sprint 2

Derek Gathright

YUI Developer

Posted: 01/31/12
  • sprint changed from sprint 2 to sprint 3

Derek Gathright

YUI Developer

Posted: 03/27/12
  • milestone changed from 3.5.0 to 3.6.0

Derek Gathright

YUI Developer

Posted: 03/27/12
  • sprint changed from sprint 3 to backlog

Derek Gathright

YUI Developer

Posted: 07/21/12
  • resolution changed to fixed
  • status changed from accepted to closed

This isn't an issue with the my latest dev code. Will be integrated into core soon. https://github.com/derek/scrollview-sandbox/tree/master/dualaxispaginator