Merge lp:~mterry/unity8/active-call-in-greeter into lp:unity8

Proposed by Michael Terry
Status: Merged
Approved by: Nick Dedekind
Approved revision: 1279
Merged at revision: 1386
Proposed branch: lp:~mterry/unity8/active-call-in-greeter
Merge into: lp:unity8
Prerequisite: lp:~mterry/unity8/emergency-dialer-fix
Diff against target: 56 lines (+8/-2)
3 files modified
qml/Components/Lockscreen.qml (+2/-1)
qml/Panel/ActiveCallHint.qml (+3/-1)
qml/Shell.qml (+3/-0)
To merge this branch: bzr merge lp:~mterry/unity8/active-call-in-greeter
Reviewer Review Type Date Requested Status
Nick Dedekind (community) Approve
PS Jenkins bot (community) continuous-integration Needs Fixing
Review via email: mp+237690@code.launchpad.net

Commit message

When greeter or lockscreen has focus, show active call panel.

Description of the change

When greeter or lockscreen has focus, show active call panel.

== Checklist ==

 * Are there any related MPs required for this MP to build/function as expected? Please list.
 Yes, the prerequisite branch

 * Did you perform an exploratory manual test run of your code change and any related functionality?
 Yes

 * Did you make sure that your branch does not contain spurious tags?
 Yes

 * If you changed the packaging (debian), did you subscribe the ubuntu-unity team to this MP?
 NA

 * If you changed the UI, has there been a design review?
 NA

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
1279. By Michael Terry

Switch text of emergency call button when in a call

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Nick Dedekind (nick-dedekind) wrote :

 * Did you perform an exploratory manual test run of the code change and any related functionality?
Yes

 * Did CI run pass? If not, please explain why.
Not related

 * Did you make sure that the branch does not contain spurious tags?
yes

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'qml/Components/Lockscreen.qml'
2--- qml/Components/Lockscreen.qml 2014-10-07 08:19:36 +0000
3+++ qml/Components/Lockscreen.qml 2014-10-09 12:52:28 +0000
4@@ -17,6 +17,7 @@
5 import QtQuick 2.0
6 import Ubuntu.Components 1.0
7 import Ubuntu.Components.Popups 1.0
8+import Ubuntu.Telephony 0.1 as Telephony
9
10 Showable {
11 id: root
12@@ -217,7 +218,7 @@
13 objectName: "emergencyCallLabel"
14 anchors.horizontalCenter: parent.horizontalCenter
15
16- text: i18n.tr("Emergency Call")
17+ text: callManager.hasCalls ? i18n.tr("Return to Call") : i18n.tr("Emergency Call")
18 color: "#f3f3e7"
19 }
20
21
22=== modified file 'qml/Panel/ActiveCallHint.qml'
23--- qml/Panel/ActiveCallHint.qml 2014-09-11 00:57:20 +0000
24+++ qml/Panel/ActiveCallHint.qml 2014-10-09 12:52:28 +0000
25@@ -23,6 +23,8 @@
26 Item {
27 id: callHint
28
29+ property bool greeterShown: false
30+
31 readonly property bool active: {
32 var application = ApplicationManager.findApplication("dialer-app");
33
34@@ -33,7 +35,7 @@
35 return application.state !== ApplicationInfoInterface.Starting;
36 }
37 }
38- if (ApplicationManager.focusedApplicationId !== "dialer-app") {
39+ if (greeterShown || ApplicationManager.focusedApplicationId !== "dialer-app") {
40 if (application) {
41 // Don't show if application is still starting; might get a fleeting true/false.
42 return application.state !== ApplicationInfoInterface.Starting && callManager.hasCalls;
43
44=== modified file 'qml/Shell.qml'
45--- qml/Shell.qml 2014-10-09 12:52:28 +0000
46+++ qml/Shell.qml 2014-10-09 12:52:28 +0000
47@@ -640,6 +640,9 @@
48 width: parent.width > units.gu(60) ? units.gu(40) : parent.width
49 panelHeight: units.gu(3)
50 }
51+ callHint {
52+ greeterShown: greeter.shown || lockscreen.shown
53+ }
54
55 property bool topmostApplicationIsFullscreen:
56 ApplicationManager.focusedApplicationId &&

Subscribers

People subscribed via source and target branches