Opened 9 years ago
Closed 5 years ago
#15527 closed defect (patchwelcome)
Swapview bounce temporarily switches content during animation (on iPad)
Reported by: | schade | Owned by: | Patrick Ruzand |
---|---|---|---|
Priority: | undecided | Milestone: | 1.13 |
Component: | DojoX Mobile | Version: | 1.7.2 |
Keywords: | Cc: | ||
Blocked By: | Blocking: |
Description
When using a swapview like on this demo page:
http://demos.dojotoolkit.org/demos/mobileSlideShow/demo.html
The content of the views are replaced during a bounce animation. This is pretty annoying and results in a strange flickering.
It is pretty easy to reproduce on the demo page, a bit harder to explain. Here we go.
1) navigate to the page on your iPad
2) swipe to the second picture (titled "May 3, 2011")
3) this is the tricky part:
3.1) hold your finger down on the ipad
3.2) move finger to the left about 2 centimers
3.3) flick to the right (move finger fast to the right and release)
The picture now swipes to the right, a bit to far and then bounches back.
This is the expected behaviour, but, during the animation the picture is changed to the first picture (the blue napoleon fish) and back again for 0.3 seconds. A short flickr.
This does not happen if you just move your finger a bit to the left and then release - the flick is required.
I tried to debug it but I don't know swapview and scrollable well enough in order to detect the error.
What I have found out so far:
It only happends if this._bounce is true in the method onFlickAnimationEnd in scrollable.js
The problem is the animation set in the code block that checks this value in the method onFlickAnimationEnd. The code starts a delayed animation using (about line 395)
setTimeout(function(){
_this.slideTo(bounce, 0.3, "ease-out");
}, 0);
Hint: if you change the animation speed to 10 instead of 0.3 its easier for the human eye to see the error.
Change History (2)
comment:1 Changed 8 years ago by
Owner: | changed from Eric Durocher to Patrick Ruzand |
---|---|
Status: | new → assigned |
comment:2 Changed 5 years ago by
Milestone: | tbd → 1.12 |
---|---|
Resolution: | → patchwelcome |
Status: | assigned → closed |
Given that no one has shown interest in creating a patch in the past 3+ years, I'm closing this as patchwelcome.