Merge lp:~vthompson/ubuntu-weather-app/reboot-time-of-day into lp:ubuntu-weather-app

Proposed by Victor Thompson
Status: Merged
Approved by: Nekhelesh Ramananthan
Approved revision: 54
Merged at revision: 54
Proposed branch: lp:~vthompson/ubuntu-weather-app/reboot-time-of-day
Merge into: lp:ubuntu-weather-app
Diff against target: 21 lines (+3/-1)
1 file modified
app/components/HomeHourly.qml (+3/-1)
To merge this branch: bzr merge lp:~vthompson/ubuntu-weather-app/reboot-time-of-day
Reviewer Review Type Date Requested Status
Ubuntu Phone Apps Jenkins Bot continuous-integration Approve
Nekhelesh Ramananthan (community) Approve
Review via email: mp+261921@code.launchpad.net

Commit message

Show am/pm text depending upon locale

Description of the change

Show am/pm text depending upon locale

To post a comment you must log in.
Revision history for this message
Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
Nekhelesh Ramananthan (nik90) wrote :

Added a small inline comment.

review: Needs Fixing
54. By Victor Thompson

modify text string

Revision history for this message
Victor Thompson (vthompson) wrote :

I've updated the formatting of the text string.

Revision history for this message
Nekhelesh Ramananthan (nik90) wrote :

lgtm!

review: Approve
Revision history for this message
Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) wrote :
review: Approve (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'app/components/HomeHourly.qml'
--- app/components/HomeHourly.qml 2015-06-18 01:42:03 +0000
+++ app/components/HomeHourly.qml 2015-06-21 16:11:15 +0000
@@ -28,6 +28,8 @@
28 model: forecasts.length28 model: forecasts.length
29 orientation: ListView.Horizontal29 orientation: ListView.Horizontal
3030
31 property string currentDate: Qt.formatTime(new Date())
32
31 onVisibleChanged: {33 onVisibleChanged: {
32 if(visible) {34 if(visible) {
33 ListView.model = forecasts.length35 ListView.model = forecasts.length
@@ -60,7 +62,7 @@
60 anchors.horizontalCenter: parent.horizontalCenter62 anchors.horizontalCenter: parent.horizontalCenter
61 fontSize: "small"63 fontSize: "small"
62 font.weight: Font.Light64 font.weight: Font.Light
63 text: formatTimestamp(hourData.date, 'ddd')+" "+formatTime(hourData.date, 'h:mm')65 text: currentDate.search(Qt.locale().amText) !== -1 || currentDate.search(Qt.locale().pmText) !== -1 ? "%1 %2".arg(formatTimestamp(hourData.date, 'ddd')).arg(formatTime(hourData.date, 'hap')) : "%1 %2".arg(formatTimestamp(hourData.date, 'ddd')).arg(formatTime(hourData.date, 'h:mm'))
64 }66 }
6567
66 Item {68 Item {

Subscribers

People subscribed via source and target branches

to all changes: