added a comment on why animateX can't be a simple binding
commit0ec11a95492db63bcc1dd8297bfea481bbe2c990
authorMichael Zanetti <michael.zanetti@canonical.com>
dateTue 2 Sep 2014, 16:57 (12 years ago)
parent3163610
qml/Stages/PhoneStage.qml+2
@@ -44,6 +44,8 @@ Rectangle {
4444 }
4545
4646 onInverseProgressChanged: {
47+ // This can't be a simple binding because that would be triggered after this handler
48+ // while we need it active before doing the anition left/right
4749 priv.animateX = (inverseProgress == 0)
4850 if (inverseProgress == 0 && priv.oldInverseProgress > 0) {
4951 // left edge drag released. Minimum distance is given by design.
qml/Stages/TabletStage.qml+2
@@ -36,6 +36,8 @@ Rectangle {
3636 property real inverseProgress: 0 // This is the progress for left edge drags, in pixels.
3737
3838 onInverseProgressChanged: {
39+ // This can't be a simple binding because that would be triggered after this handler
40+ // while we need it active before doing the anition left/right
3941 spreadView.animateX = (inverseProgress == 0)
4042 if (inverseProgress == 0 && priv.oldInverseProgress > 0) {
4143 // left edge drag released. Minimum distance is given by design.