Merge lp:~nik90/ubuntu-clock-app/ota-10-design-polish into lp:ubuntu-clock-app

Proposed by Nekhelesh Ramananthan
Status: Merged
Merged at revision: 465
Proposed branch: lp:~nik90/ubuntu-clock-app/ota-10-design-polish
Merge into: lp:ubuntu-clock-app
Diff against target: 325 lines (+40/-29)
14 files modified
app/clock/ClockPage.qml (+3/-3)
app/components/ActionIcon.qml (+2/-2)
app/components/ClockCircle.qml (+5/-5)
app/components/DigitalMode.qml (+2/-2)
app/components/EmptyState.qml (+2/-2)
app/components/HeaderNavigation.qml (+2/-2)
app/stopwatch/StopwatchFace.qml (+2/-2)
app/stopwatch/StopwatchPage.qml (+1/-1)
app/upstreamcomponents/FastScroll.qml (+3/-3)
app/worldclock/AddWorldCityButton.qml (+1/-1)
app/worldclock/UserWorldCityDelegate.qml (+3/-3)
app/worldclock/WorldCityList.qml (+8/-1)
debian/changelog (+4/-0)
manifest.json.in (+2/-2)
To merge this branch: bzr merge lp:~nik90/ubuntu-clock-app/ota-10-design-polish
Reviewer Review Type Date Requested Status
Michal Predotka Approve
Review via email: mp+289699@code.launchpad.net

Commit message

- Fixed world city list item vertical centering
- Replaced hex color codes with UbuntuColors counterparts
- Updated framework to ubuntu-sdk-15.04.4 and base version to 3.8

Description of the change

- Fixed world city list item vertical centering
- Replaced hex color codes with UbuntuColors counterparts
- Updated framework to ubuntu-sdk-15.04.4 and base version to 3.8

The ubuntu-sdk-15.04.4 framework will ensure that the clock update reaches only OTA-10 users thereby also justifying the move to using the new UbuntuColors palette.

To post a comment you must log in.
468. By Nekhelesh Ramananthan

Updated framework to ubuntu-sdk-15.04.4 and base version to 3.8

Revision history for this message
Michal Predotka (mpredotka) wrote :

Judging by the code changes, everything looks fine. As I only run "stable" OTAs on my phone, I can't test it there.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'app/clock/ClockPage.qml'
2--- app/clock/ClockPage.qml 2016-03-02 12:04:48 +0000
3+++ app/clock/ClockPage.qml 2016-03-21 19:10:54 +0000
4@@ -208,7 +208,7 @@
5 text: clock.localizedDateString
6 textSize: Label.Medium
7 opacity: 0
8- color: "#5D5D5D"
9+ color: UbuntuColors.slate
10 }
11
12 Row {
13@@ -236,7 +236,7 @@
14 id: location
15 objectName: "location"
16
17- color: "#5D5D5D"
18+ color: UbuntuColors.slate
19 anchors.verticalCenter: locationIcon.verticalCenter
20
21 text: {
22@@ -257,7 +257,7 @@
23 id: divider
24 width: parent.width
25 height: 1
26- color: "#cdcdcd"
27+ color: UbuntuColors.silk
28 anchors {
29 top: locationRow.bottom
30 topMargin: units.gu(3)
31
32=== modified file 'app/components/ActionIcon.qml'
33--- app/components/ActionIcon.qml 2016-02-29 12:37:51 +0000
34+++ app/components/ActionIcon.qml 2016-03-21 19:10:54 +0000
35@@ -30,7 +30,7 @@
36 Rectangle {
37 visible: abstractButton.pressed
38 anchors.fill: parent
39- color: "#CDCDCD" // #TODO: Replace with UbuntuColors.Silk after OTA-10
40+ color: UbuntuColors.silk
41 }
42
43 Icon {
44@@ -38,6 +38,6 @@
45 width: units.gu(2)
46 height: width
47 anchors.centerIn: parent
48- color: "#5d5d5d"
49+ color: UbuntuColors.slate
50 }
51 }
52
53=== modified file 'app/components/ClockCircle.qml'
54--- app/components/ClockCircle.qml 2016-02-25 22:16:54 +0000
55+++ app/components/ClockCircle.qml 2016-03-21 19:10:54 +0000
56@@ -29,7 +29,7 @@
57
58 property bool isFoldVisible: true
59
60- color: "#f7f7f7"
61+ color: UbuntuColors.porcelain
62 borderWidth: units.dp(1)
63 borderColorTop: "#00000000"
64 borderColorBottom: "#6E6E6E"
65@@ -42,10 +42,10 @@
66 anchors.margins: borderWidth
67 radius: height / 2
68 gradient: Gradient {
69- GradientStop { position: 0.0; color: "#f7f7f7" }
70- GradientStop { position: 0.5; color: "#f7f7f7" }
71- GradientStop { position: 0.5; color: "#fdfdfd" }
72- GradientStop { position: 1.0; color: "#fdfdfd" }
73+ GradientStop { position: 0.0; color: UbuntuColors.porcelain }
74+ GradientStop { position: 0.5; color: UbuntuColors.porcelain }
75+ GradientStop { position: 0.5; color: "#FDFDFD" }
76+ GradientStop { position: 1.0; color: "#FDFDFD" }
77 }
78 }
79 }
80
81=== modified file 'app/components/DigitalMode.qml'
82--- app/components/DigitalMode.qml 2016-03-02 12:04:48 +0000
83+++ app/components/DigitalMode.qml 2016-03-21 19:10:54 +0000
84@@ -48,7 +48,7 @@
85
86 anchors.centerIn: parent
87
88- color: "#5D5D5D"
89+ color: UbuntuColors.slate
90 font.pixelSize: maxTimeFontSize
91
92 text: {
93@@ -75,7 +75,7 @@
94
95 font.pixelSize: maxPeriodFontSize
96 visible: text !== ""
97- color: "#5D5D5D"
98+ color: UbuntuColors.slate
99 text: {
100 if (localizedTimeString.search(Qt.locale().amText) !== -1) {
101 // 12 hour format detected with the localised AM text
102
103=== modified file 'app/components/EmptyState.qml'
104--- app/components/EmptyState.qml 2016-02-28 20:43:39 +0000
105+++ app/components/EmptyState.qml 2016-03-21 19:10:54 +0000
106@@ -51,7 +51,7 @@
107 anchors.top: emptyIcon.bottom
108 anchors.topMargin: units.gu(4)
109 horizontalAlignment: Text.AlignHCenter
110- color: "#5d5d5d"
111+ color: UbuntuColors.slate
112 }
113
114 Label {
115@@ -60,6 +60,6 @@
116 wrapMode: Text.WordWrap
117 anchors.top: emptyLabel.bottom
118 horizontalAlignment: Text.AlignHCenter
119- color: "#888888"
120+ color: UbuntuColors.ash
121 }
122 }
123
124=== modified file 'app/components/HeaderNavigation.qml'
125--- app/components/HeaderNavigation.qml 2016-03-03 10:37:15 +0000
126+++ app/components/HeaderNavigation.qml 2016-03-21 19:10:54 +0000
127@@ -35,7 +35,7 @@
128 objectName: "clockNavigationButton"
129
130 icon.name: "clock"
131- icon.color: listview.currentIndex === 0 ? "#19b6ee" : "#5d5d5d" // #TODO: Replace with UbuntuColors.Blue and UbuntuColors.Slate after OTA-10
132+ icon.color: listview.currentIndex === 0 ? UbuntuColors.blue : UbuntuColors.slate
133 onClicked: listview.moveToClockPage()
134 }
135
136@@ -44,7 +44,7 @@
137 objectName: "stopwatchNavigationButton"
138
139 icon.name: "stopwatch"
140- icon.color: listview.currentIndex === 1 ? "#19b6ee" : "#5d5d5d" // #TODO: Replace with UbuntuColors.Blue and UbuntuColors.Slate after OTA-10
141+ icon.color: listview.currentIndex === 1 ? UbuntuColors.blue : UbuntuColors.slate
142 onClicked: listview.moveToStopwatchPage()
143 }
144 }
145
146=== modified file 'app/stopwatch/StopwatchFace.qml'
147--- app/stopwatch/StopwatchFace.qml 2016-03-03 10:37:15 +0000
148+++ app/stopwatch/StopwatchFace.qml 2016-03-21 19:10:54 +0000
149@@ -40,7 +40,7 @@
150 text: stopwatchFormatTime.millisToTimeString(milliseconds, true)
151 font.pixelSize: units.dp(36)
152 anchors.centerIn: parent
153- color: "#5D5D5D"
154+ color: UbuntuColors.slate
155 }
156
157 Label {
158@@ -49,7 +49,7 @@
159
160 text: stopwatchFormatTime.millisToString(milliseconds)
161 textSize: Label.Large
162- color: "#5D5D5D"
163+ color: UbuntuColors.slate
164 anchors {
165 top: time.bottom
166 topMargin: units.gu(1.5)
167
168=== modified file 'app/stopwatch/StopwatchPage.qml'
169--- app/stopwatch/StopwatchPage.qml 2016-03-03 14:38:55 +0000
170+++ app/stopwatch/StopwatchPage.qml 2016-03-21 19:10:54 +0000
171@@ -101,7 +101,7 @@
172 width: parent.width / 2 - units.gu(1)
173 height: units.gu(4)
174 x: stopwatchEngine.previousTimeOfStopwatch !== 0 || stopwatchEngine.running ? parent.width - width : parent.width
175- color: "#f7f7f7"
176+ color: UbuntuColors.porcelain
177 visible: stopwatchEngine.previousTimeOfStopwatch !== 0 || stopwatchEngine.running
178 onClicked: {
179 if (stopwatchEngine.running) {
180
181=== modified file 'app/upstreamcomponents/FastScroll.qml'
182--- app/upstreamcomponents/FastScroll.qml 2016-02-29 12:37:51 +0000
183+++ app/upstreamcomponents/FastScroll.qml 2016-03-21 19:10:54 +0000
184@@ -83,7 +83,7 @@
185 id: magnified
186
187 aspect: UbuntuShape.Flat
188- backgroundColor: "#3B3B3B" // #TODO: Replace with UbuntuColors.Inkstone after OTA-10
189+ backgroundColor: UbuntuColors.inkstone
190 radius: "large"
191 height: pinSize * 2
192 width: height
193@@ -122,7 +122,7 @@
194 height: pinSize
195 width: height
196 aspect: UbuntuShape.Flat
197- backgroundColor: "#3B3B3B" // #TODO: Replace with UbuntuColors.Inkstone after OTA-10
198+ backgroundColor: UbuntuColors.inkstone
199 opacity: rail.opacity
200 x: rail.x
201 y: {
202@@ -183,7 +183,7 @@
203 horizontalAlignment: Text.AlignHCenter
204 text: modelData
205 textSize: Label.XSmall
206- color: cursor.y === y ? "white" : "#3B3B3B" // #TODO: Replace with UbuntuColors.Inkstone after OTA-10
207+ color: cursor.y === y ? "white" : UbuntuColors.inkstone
208 opacity: !internal.modelDirty && Sections.contains(text) ? 1.0 : 0.5
209 }
210 }
211
212=== modified file 'app/worldclock/AddWorldCityButton.qml'
213--- app/worldclock/AddWorldCityButton.qml 2016-02-29 12:37:51 +0000
214+++ app/worldclock/AddWorldCityButton.qml 2016-03-21 19:10:54 +0000
215@@ -48,7 +48,7 @@
216
217 Icon {
218 anchors.centerIn: parent
219- color: "#3B3B3B" // #TODO: Replace with UbuntuColors.Inkstone after OTA-10
220+ color: UbuntuColors.inkstone
221 name: "add"
222 height: units.gu(2)
223 width: height
224
225=== modified file 'app/worldclock/UserWorldCityDelegate.qml'
226--- app/worldclock/UserWorldCityDelegate.qml 2016-03-02 12:04:48 +0000
227+++ app/worldclock/UserWorldCityDelegate.qml 2016-03-21 19:10:54 +0000
228@@ -59,7 +59,7 @@
229 textSize: Label.Medium
230 width: parent.width
231 elide: Text.ElideRight
232- color: "#5d5d5d"
233+ color: UbuntuColors.slate
234 }
235
236 Label {
237@@ -68,7 +68,7 @@
238 textSize: Label.Medium
239 width: parent.width
240 elide: Text.ElideRight
241- color: "#888888"
242+ color: UbuntuColors.ash
243 }
244 }
245
246@@ -122,7 +122,7 @@
247 anchors.right: parent.right
248 anchors.verticalCenter: parent.verticalCenter
249
250- color: "#5d5d5d"
251+ color: UbuntuColors.slate
252 textSize: Label.Medium
253 wrapMode: Text.WordWrap
254 maximumLineCount: 2
255
256=== modified file 'app/worldclock/WorldCityList.qml'
257--- app/worldclock/WorldCityList.qml 2016-03-02 19:15:40 +0000
258+++ app/worldclock/WorldCityList.qml 2016-03-21 19:10:54 +0000
259@@ -128,7 +128,7 @@
260 var url = String("%1%2%3")
261 .arg("http://geoname-lookup.ubuntu.com/?query=")
262 .arg(searchField.text)
263- .arg("&app=com.ubuntu.clock&version=3.7.x")
264+ .arg("&app=com.ubuntu.clock&version=3.8.x")
265 console.log("Online URL: " + url)
266 if (jsonTimeZoneModelLoader.status === Loader.Ready) {
267 jsonTimeZoneModel.source = Qt.resolvedUrl(url)
268@@ -330,13 +330,20 @@
269 delegate: ListItem {
270 divider.visible: false
271 objectName: "defaultWorldCityItem" + index
272+ height: worldCityListItemLayout.height
273+
274 ListItemLayout {
275+ id: worldCityListItemLayout
276+
277 title.text: cityName
278 title.objectName: "defaultCityNameText"
279 subtitle.text: countryName
280 subtitle.textSize: Label.Medium
281 subtitle.objectName: "defaultCountryNameText"
282
283+ padding.top: units.gu(1.5)
284+ padding.bottom: units.gu(1.5)
285+
286 Label {
287 id: _localTime
288 text: localizedZoneTime
289
290=== modified file 'debian/changelog'
291--- debian/changelog 2016-03-21 18:38:43 +0000
292+++ debian/changelog 2016-03-21 19:10:54 +0000
293@@ -7,6 +7,10 @@
294 * Added Stopwatch autopilot tests for adding/deleting laps, checking
295 start, pause and stop stopwatch states. (LP: #1490206)
296 * Moved laps list delegate into its own file
297+ * Fixed world city listitem vertical centering.
298+ * Replaced hex color codes with UbuntuColors counterparts.
299+ * Updated framework to ubuntu-sdk-15.04.4 and incremented base version to 3.8
300+ to prevent clock app release to OTA-9.1 users.
301
302 [Ken VanDine]
303 * Bump Ubuntu.Content import to 1.3 to fix theme issue (LP: #1554621)
304
305=== modified file 'manifest.json.in'
306--- manifest.json.in 2016-02-26 12:28:26 +0000
307+++ manifest.json.in 2016-03-21 19:10:54 +0000
308@@ -1,7 +1,7 @@
309 {
310 "name": "com.ubuntu.clock",
311 "description": "A sophisticated clock app for Ubuntu Touch",
312- "framework": "ubuntu-sdk-15.04.3",
313+ "framework": "ubuntu-sdk-15.04.4",
314 "architecture": "@CLICK_ARCH@",
315 "title": "Clock",
316 "icon": "clock@30.png",
317@@ -12,7 +12,7 @@
318 "urls": "share/url-dispatcher/urls/com.ubuntu.clock_clock.url-dispatcher"
319 }
320 },
321- "version": "3.7.@BZR_REVNO@",
322+ "version": "3.8.@BZR_REVNO@",
323 "maintainer": "Ubuntu App Cats <ubuntu-touch-coreapps@lists.launchpad.net>",
324 "x-test": {
325 "autopilot": {

Subscribers

People subscribed via source and target branches