Opened 7 years ago

Closed 4 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 7 years ago by Patrick Ruzand

Owner: changed from Eric Durocher to Patrick Ruzand
Status: newassigned

comment:2 Changed 4 years ago by dylan

Milestone: tbd1.12
Resolution: patchwelcome
Status: assignedclosed

Given that no one has shown interest in creating a patch in the past 3+ years, I'm closing this as patchwelcome.

Note: See TracTickets for help on using tickets.