Switch toolbar animation to opacity
authorMohammed Sameer <msameer@foolab.org>
Sun, 25 Aug 2013 17:41:50 +0000 (20:41 +0300)
committerMohammed Sameer <msameer@foolab.org>
Sun, 25 Aug 2013 17:52:33 +0000 (20:52 +0300)
This also avoids 'Binding loop detected for property "height"' warning

qml/PostCaptureView.qml
qml/VideoPlayerPage.qml

index 0b18824..52eae74 100644 (file)
@@ -24,8 +24,6 @@ import QtQuick 2.0
 import CameraPlus 1.0
 import QtCamera 1.0
 
-// TODO: qrc:/qml/PostCaptureView.qml:104:5: QML CameraToolBar: Binding loop detected for property "height"
-
 Item {
     id: postCaptureView
 
@@ -108,18 +106,18 @@ Item {
         expanded: true
         hideBack: true
         anchors.bottom: parent.bottom
-        anchors.bottomMargin: show ? 20 : -1 * (height + 20)
+        anchors.bottomMargin: 20
         anchors.left: parent.left
         anchors.leftMargin: 20
         anchors.right: parent.right
         anchors.rightMargin: 20
-        opacity: 0.8
-
+        opacity: show ? 0.8 : 0.0
+        visible: opacity > 0
         property bool show: deleteDialog.isOpen || deleteDialog.isOpening ||
             hideTimer.running ||
             (view.currentItem != null && view.currentItem.error) && !view.currentItem.playing
 
-        Behavior on anchors.bottomMargin {
+        Behavior on opacity {
             PropertyAnimation { duration: view.currentItem && view.currentItem.playing ? 0 : 200 }
         }
 
@@ -174,7 +172,7 @@ Item {
     Rectangle {
         opacity: toolBar.opacity
         anchors.top: parent.top
-        anchors.topMargin: toolBar.show ? 20 : -1 * (height + 20)
+        anchors.topMargin: 20
         anchors.left: parent.left
         anchors.leftMargin: 20
         anchors.right: parent.right
@@ -185,10 +183,6 @@ Item {
         border.color: toolBar.border.color
         radius: toolBar.radius
 
-        Behavior on anchors.topMargin {
-            PropertyAnimation { duration: view.currentItem && view.currentItem.playing ? 0 : 200 }
-        }
-
         Flow {
             width: parent.width - 40
             x: 20
index 7f4d981..2e310bf 100644 (file)
@@ -94,12 +94,13 @@ Item {
         hideBack: true
         expanded: true
         anchors.bottom: parent.bottom
-        anchors.bottomMargin: show ? 20 : -1 * (height + 20)
+        anchors.bottomMargin: 20
         anchors.left: parent.left
         anchors.leftMargin: 20
-        opacity: 0.5
+        opacity: show ? 0.8 : 0.0
+        visible: opacity > 0
 
-        Behavior on anchors.bottomMargin {
+        Behavior on opacity {
             PropertyAnimation { duration: 200; }
         }