[Merge] lp:~boiko/dialer-app/active_call_indicator into lp:dialer-app

Nick Dedekind nick.dedekind at canonical.com
Wed Jun 25 16:44:15 UTC 2014


I took a look into it, and think this works pretty well:

=== modified file 'src/qml/LiveCallPage/LiveCall.qml'
--- src/qml/LiveCallPage/LiveCall.qml	2014-06-19 20:42:19 +0000
+++ src/qml/LiveCallPage/LiveCall.qml	2014-06-25 16:43:32 +0000
@@ -120,12 +120,10 @@
         dtmfVisible = (call && call.voicemail);
     }
 
-    onActiveChanged: {
-        callManager.callIndicatorVisible = !active;
-    }
-
-    Component.onCompleted: {
-        callManager.callIndicatorVisible = !active;
+    Binding {
+        target: callManager
+        property: "callIndicatorVisible"
+        value: !active || !Qt.application.active
     }
 
     Timer {


-- 
https://code.launchpad.net/~boiko/dialer-app/active_call_indicator/+merge/223825
Your team Ubuntu Phablet Team is subscribed to branch lp:dialer-app.



More information about the Ubuntu-reviews mailing list