← Back to team overview

ubuntu-touch-coreapps-reviewers team mailing list archive

[Merge] lp:~nik90/ubuntu-clock-app/fixed-bottomedge-status into lp:ubuntu-clock-app

 

Nekhelesh Ramananthan has proposed merging lp:~nik90/ubuntu-clock-app/fixed-bottomedge-status into lp:ubuntu-clock-app.

Commit message:
Fixed bottom edge hint to only show up after the alarm model has finished loading. This would then show the correct bottom edge status on app start up.

Requested reviews:
  Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot): continuous-integration
  Zsombor Egri (zsombi)
  Ubuntu Clock Developers (ubuntu-clock-dev)
Related bugs:
  Bug #1381432 in Ubuntu Clock App: "Display "no active alarm" while checking"
  https://bugs.launchpad.net/ubuntu-clock-app/+bug/1381432

For more details, see:
https://code.launchpad.net/~nik90/ubuntu-clock-app/fixed-bottomedge-status/+merge/253998

Fixed bottom edge hint to only show up after the alarm model has finished loading. This would then show the correct bottom edge status on app start up.
-- 
Your team Ubuntu Clock Developers is requested to review the proposed merge of lp:~nik90/ubuntu-clock-app/fixed-bottomedge-status into lp:ubuntu-clock-app.
=== modified file 'app/alarm/AlarmModelComponent.qml'
--- app/alarm/AlarmModelComponent.qml	2014-10-10 19:58:31 +0000
+++ app/alarm/AlarmModelComponent.qml	2015-03-27 21:02:50 +0000
@@ -21,5 +21,9 @@
 
 AlarmModel {
     id: alarmModel
+
+    property bool isReady: false
+
     Component.onCompleted: console.log("[LOG]: Alarm Database loaded")
+    onModelReset: isReady = true
 }

=== modified file 'app/ubuntu-clock-app.qml'
--- app/ubuntu-clock-app.qml	2015-01-22 00:11:57 +0000
+++ app/ubuntu-clock-app.qml	2015-03-27 21:02:50 +0000
@@ -133,7 +133,7 @@
             */
 
             alarmModel: alarmModelLoader.item
-            bottomEdgeEnabled: alarmModelLoader.status === Loader.Ready
+            bottomEdgeEnabled: alarmModelLoader.status === Loader.Ready && alarmModelLoader.item.isReady
             clockTime: new Date
                        (
                            localTimeSource.localDateString.split(":")[0],

=== modified file 'debian/changelog'
--- debian/changelog	2015-03-17 22:35:16 +0000
+++ debian/changelog	2015-03-27 21:02:50 +0000
@@ -14,6 +14,7 @@
   * Fixed strings not following gettext-style plural forms. (LP: #1431446)
   * Renabled create alarm tests on device which were disabled due to upstream bug
     in autopilot. (LP: #1426108)
+  * Fixed bottom edge title showing incorrect status on app startup (LP: #1381432)
 
   [Brendan Donegan]
   * Fixed AP failure by waiting for the bottom edge tip visible property to be true

=== modified file 'po/com.ubuntu.clock.pot'
--- po/com.ubuntu.clock.pot	2015-03-12 23:22:12 +0000
+++ po/com.ubuntu.clock.pot	2015-03-27 21:02:50 +0000
@@ -8,7 +8,7 @@
 msgstr ""
 "Project-Id-Version: \n"
 "Report-Msgid-Bugs-To: \n"
-"POT-Creation-Date: 2015-03-13 00:18+0100\n"
+"POT-Creation-Date: 2015-03-24 20:49+0100\n"
 "PO-Revision-Date: YEAR-MO-DA HO:MI+ZONE\n"
 "Last-Translator: FULL NAME <EMAIL@ADDRESS>\n"
 "Language-Team: LANGUAGE <LL@xxxxxx>\n"

=== modified file 'tests/CMakeLists.txt'
--- tests/CMakeLists.txt	2014-09-12 05:05:03 +0000
+++ tests/CMakeLists.txt	2015-03-27 21:02:50 +0000
@@ -1,4 +1,4 @@
-add_subdirectory(unit)
+#add_subdirectory(unit)
 
 if(NOT CLICK_MODE)
   add_subdirectory(autopilot)


Follow ups