Merge lp:~ahayzen/ubuntu-weather-app/reboot-uc1.3-bump into lp:ubuntu-weather-app
- reboot-uc1.3-bump
- Merge into reboot
Status: | Merged | ||||
---|---|---|---|---|---|
Approved by: | Victor Thompson | ||||
Approved revision: | 166 | ||||
Merged at revision: | 180 | ||||
Proposed branch: | lp:~ahayzen/ubuntu-weather-app/reboot-uc1.3-bump | ||||
Merge into: | lp:ubuntu-weather-app | ||||
Diff against target: |
776 lines (+93/-67) 42 files modified
app/components/CurrentLocation.qml (+1/-1) app/components/DayDelegate.qml (+1/-1) app/components/DayDelegateExtraInfo.qml (+1/-1) app/components/ExpandableListItem.qml (+1/-1) app/components/FakeHeader.qml (+4/-4) app/components/FastScroll.qml (+1/-1) app/components/ForecastDetailsDelegate.qml (+1/-1) app/components/HeaderRow.qml (+1/-1) app/components/HomeGraphic.qml (+1/-1) app/components/HomeHourly.qml (+1/-1) app/components/HomePageEmptyStateComponent.qml (+1/-1) app/components/HomeTempInfo.qml (+1/-1) app/components/ListItemActions/CheckBox.qml (+1/-1) app/components/ListItemActions/Remove.qml (+1/-1) app/components/ListItemReorderComponent.qml (+1/-1) app/components/ListItemWithActions.qml (+1/-1) app/components/LoadingIndicator.qml (+1/-1) app/components/LocationsPageEmptyStateComponent.qml (+1/-1) app/components/MultiSelectHeadState.qml (+1/-1) app/components/MultiSelectListView.qml (+1/-1) app/components/NetworkErrorStateComponent.qml (+1/-1) app/components/NoAPIKeyErrorStateComponent.qml (+1/-1) app/components/PageWithBottomEdge.qml (+21/-10) app/components/SettingsButton.qml (+1/-1) app/components/StandardListItem.qml (+1/-1) app/components/WeatherListItem.qml (+1/-1) app/components/WeatherListView.qml (+1/-1) app/ubuntu-weather-app.qml (+1/-1) app/ui/AddLocationPage.qml (+8/-6) app/ui/HomePage.qml (+7/-1) app/ui/LocationPane.qml (+1/-1) app/ui/LocationsPage.qml (+2/-2) app/ui/SettingsPage.qml (+1/-1) app/ui/settings/DataProviderPage.qml (+1/-1) app/ui/settings/LocationPage.qml (+1/-1) app/ui/settings/RefreshIntervalPage.qml (+1/-1) app/ui/settings/UnitsPage.qml (+1/-1) debian/changelog (+1/-0) manifest.json.in (+1/-1) po/com.ubuntu.weather.pot (+6/-6) tests/autopilot/ubuntu_weather_app/__init__.py (+10/-5) ubuntu-weather-app.desktop.in.in (+1/-0) |
||||
To merge this branch: | bzr merge lp:~ahayzen/ubuntu-weather-app/reboot-uc1.3-bump | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Victor Thompson | Approve | ||
Jenkins Bot | continuous-integration | Approve | |
Bartosz Kosiorek (community) | Approve | ||
Ubuntu Phone Apps Jenkins Bot | continuous-integration | Approve | |
Andrew Hayzen | Abstain | ||
Review via email: mp+276466@code.launchpad.net |
Commit message
* Update app to use the Ubuntu Components version 1.3
Description of the change
* Update app to use the Ubuntu Components version 1.3
I think I've managed to tweak the FakeHeader and PageWithBottomEdge to match the new header height and styling requirements :-)
Please test AP locally as well.
Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) wrote : | # |
Jenkins Bot (ubuntu-core-apps-jenkins-bot) wrote : | # |
FAILED: Continuous integration, rev:154
https:/
Executed test runs:
None: https:/
Click here to trigger a rebuild:
https:/
- 155. By Andrew Hayzen
-
* Merge of trunk
- 156. By Andrew Hayzen
-
* Update changelog
Jenkins Bot (ubuntu-core-apps-jenkins-bot) wrote : | # |
PASSED: Continuous integration, rev:156
https:/
Executed test runs:
None: https:/
Click here to trigger a rebuild:
https:/
Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) wrote : | # |
PASSED: Continuous integration, rev:156
http://
Executed test runs:
SUCCESS: http://
Click here to trigger a rebuild:
http://
Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) wrote : | # |
PASSED: Continuous integration, rev:156
http://
Executed test runs:
SUCCESS: http://
Click here to trigger a rebuild:
http://
Andrew Hayzen (ahayzen) wrote : | # |
AP tests failing alot due to bottomEdge .isReady being incorrect and other issues.
- 157. By Andrew Hayzen
-
* Tweaks to bottomEdge for autopilot
Andrew Hayzen (ahayzen) wrote : | # |
This is now passing autopilot :-)
autopilot PASS
Jenkins Bot (ubuntu-core-apps-jenkins-bot) wrote : | # |
PASSED: Continuous integration, rev:157
https:/
Executed test runs:
None: https:/
Click here to trigger a rebuild:
https:/
Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) wrote : | # |
PASSED: Continuous integration, rev:157
http://
Executed test runs:
SUCCESS: http://
Click here to trigger a rebuild:
http://
Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) wrote : | # |
PASSED: Continuous integration, rev:157
http://
Executed test runs:
SUCCESS: http://
Click here to trigger a rebuild:
http://
Victor Thompson (vthompson) wrote : | # |
IMO the startup animation where the header hides and the animation of the header when swiping from the bottom edge are a bit annoying.
It would at least seem like the animation of closing the fake header has the wrong height (height of the old header) initially.
Other than that this seems mostly fine, as I can't find any actual regressions.
Bartosz Kosiorek (gang65) wrote : | # |
I would like to test weather app, but unfortunately I have an errors:
qml: Sent request URL: http://
qml: retry of http://
qml: Sent request URL: http://
qml: {"msg":"wrong response http code, got 401","request"
qml: wrong response http code, got 401 / http://
qml: {"msg":"wrong response http code, got 401","request"
According to documentation:
http://
Starting from 9 October 2015 our API requires a valid APPID for access. Note that this does not mean that our API is subscription-only now - please take a minute to register a free account to receive a key.
We are sorry for inconvenience but this is a necessary measure that will help us deliver our services to you faster and more reliably.
For FOSS developers: we welcome free and open source software and are willing to help you. If you want to use OWM data in your free software application please register an API key and file a ticket describing your application and API key registered. OWM will review your request lift access limits for your key if used in open source application.
@Victor Do you know how to resolve this issue?
Andrew Hayzen (ahayzen) wrote : | # |
@Bartosz, we now have a MP [0] which adds instructions of how to add an OWM key :-)
0 - https:/
Bartosz Kosiorek (gang65) wrote : | # |
Thanks Andrew.
I made some testing of this branch and here is what I found:
1. Missing title for Desktop. With new SDK, the title is working correctly if you hide header
title: i18n.tr("Weather")
2. There are obsolete warning:
file://
3. There are undefined types errors in Location Pane:
file://
file://
file://
file://
4. There are undefined types errors in HomeTempInfo.qml:
file://
file://
file://
file://
file://
file://
file://
file://
file://
file://
file://
file://
file://
file://
file://
file://
Bartosz Kosiorek (gang65) wrote : | # |
There are also following deprecate warnings:
file://
file://
file://
file://
- 158. By Andrew Hayzen
-
* Fix deprecate warnings and title for desktop
- 159. By Andrew Hayzen
-
* Merge of trunk
Andrew Hayzen (ahayzen) wrote : | # |
1) Added title to the HomePage.qml
2) Fixed the bottomEdge issues
3) Not been able to reproduce this one yet, could you provide exact steps
4) Same as 3)
5) These look like issues due to the old listitems (note /Ubuntu/
Meanwhile there appear to be offset issues in the LocationPage when switching on/off the auto detect location, which I believe partially existed before. I have minimised the issues when actually on the LocationPage, but it looks like I need to force the bottomEdge hint to reload when the location detection is toggled. As the offset is incorrect after switching until the app is restarted. I'll investigate this :-)
Jenkins Bot (ubuntu-core-apps-jenkins-bot) wrote : | # |
PASSED: Continuous integration, rev:159
https:/
Executed test runs:
None: https:/
Click here to trigger a rebuild:
https:/
Bartosz Kosiorek (gang65) wrote : | # |
See my inline comments.
Also after clicking on city (from city list), I have following error:
file://
Bartosz Kosiorek (gang65) wrote : | # |
@Andrew @Victor
Could you please also take a look at:
https:/
:-)
Thanks
Andrew Hayzen (ahayzen) wrote : | # |
@Bartosz, as the PageWithBottomEdge is an 'upstream' component to us, any modifications we make to the code we try to add a // CUSTOM comment next to them, and any code we remove we try to comment out rather than remove. This is to ease upgrading and help merge conflicts when changes are made upstream, so in this case we would comment out that block and add // CUSTOM as the block is upstream here [2].
Also IIRC the PageWrapperUtil
Bartosz Kosiorek (gang65) wrote : | # |
It looks perfectly for me. Thanks
Victor Thompson (vthompson) wrote : | # |
Please also bump the new NoAPIKeyErrorSt
I still see all the same odd header behaviors.
I'm not sure I agree with using "Weather" as the title. The title on the desktop did not change with this MP, so fixing it here doesn't seem necessary. I don't think the user should see "Weather" as the header is hidden/shown.
I think you need to rebuild the pot file? It seems your mp is removing some stuff.
- 160. By Andrew Hayzen
-
* Bump merged component
Jenkins Bot (ubuntu-core-apps-jenkins-bot) wrote : | # |
PASSED: Continuous integration, rev:160
https:/
Executed test runs:
None: https:/
Click here to trigger a rebuild:
https:/
- 161. By Andrew Hayzen
-
* Update .pot file
Jenkins Bot (ubuntu-core-apps-jenkins-bot) wrote : | # |
PASSED: Continuous integration, rev:161
https:/
Executed test runs:
None: https:/
Click here to trigger a rebuild:
https:/
- 162. By Andrew Hayzen
-
* Show header in splash to minimise slide up effect
Jenkins Bot (ubuntu-core-apps-jenkins-bot) wrote : | # |
PASSED: Continuous integration, rev:162
https:/
Executed test runs:
None: https:/
Click here to trigger a rebuild:
https:/
- 163. By Andrew Hayzen
-
* Remove Weather from title for now due to incorrect header animation
Jenkins Bot (ubuntu-core-apps-jenkins-bot) wrote : | # |
PASSED: Continuous integration, rev:163
https:/
Executed test runs:
None: https:/
Click here to trigger a rebuild:
https:/
- 164. By Andrew Hayzen
-
* Reduce bounce animation of content when pushing a page
Jenkins Bot (ubuntu-core-apps-jenkins-bot) wrote : | # |
PASSED: Continuous integration, rev:164
https:/
Executed test runs:
None: https:/
Click here to trigger a rebuild:
https:/
- 165. By Andrew Hayzen
-
* Fix the bottom edge hint topMargin being incorrect on the second pull
Jenkins Bot (ubuntu-core-apps-jenkins-bot) wrote : | # |
PASSED: Continuous integration, rev:165
https:/
Executed test runs:
None: https:/
Click here to trigger a rebuild:
https:/
- 166. By Andrew Hayzen
-
* Add extra code commentary
Jenkins Bot (ubuntu-core-apps-jenkins-bot) wrote : | # |
PASSED: Continuous integration, rev:166
https:/
Executed test runs:
None: https:/
Click here to trigger a rebuild:
https:/
Victor Thompson (vthompson) wrote : | # |
Thanks for the extra fixes and code commentary. I think we still should work on resolving the issues with the FakeHeader, but for now this is great... as our header/BottomEdge transition wasn't consistent with all other apps previously, so I don't mind it being somewhat "different" for the time being.
Preview Diff
1 | === modified file 'app/components/CurrentLocation.qml' |
2 | --- app/components/CurrentLocation.qml 2015-07-27 00:57:57 +0000 |
3 | +++ app/components/CurrentLocation.qml 2015-11-26 02:38:41 +0000 |
4 | @@ -19,7 +19,7 @@ |
5 | import QtLocation 5.3 |
6 | import QtPositioning 5.2 |
7 | import QtQuick 2.4 |
8 | -import Ubuntu.Components 1.2 |
9 | +import Ubuntu.Components 1.3 |
10 | import "../data/WeatherApi.js" as WeatherApi |
11 | |
12 | |
13 | |
14 | === modified file 'app/components/DayDelegate.qml' |
15 | --- app/components/DayDelegate.qml 2015-10-16 00:12:57 +0000 |
16 | +++ app/components/DayDelegate.qml 2015-11-26 02:38:41 +0000 |
17 | @@ -17,7 +17,7 @@ |
18 | */ |
19 | |
20 | import QtQuick 2.4 |
21 | -import Ubuntu.Components 1.2 |
22 | +import Ubuntu.Components 1.3 |
23 | |
24 | ListItem { |
25 | id: dayDelegate |
26 | |
27 | === modified file 'app/components/DayDelegateExtraInfo.qml' |
28 | --- app/components/DayDelegateExtraInfo.qml 2015-09-29 15:21:55 +0000 |
29 | +++ app/components/DayDelegateExtraInfo.qml 2015-11-26 02:38:41 +0000 |
30 | @@ -17,7 +17,7 @@ |
31 | */ |
32 | |
33 | import QtQuick 2.4 |
34 | -import Ubuntu.Components 1.2 |
35 | +import Ubuntu.Components 1.3 |
36 | |
37 | |
38 | Column { |
39 | |
40 | === modified file 'app/components/ExpandableListItem.qml' |
41 | --- app/components/ExpandableListItem.qml 2015-06-18 01:42:03 +0000 |
42 | +++ app/components/ExpandableListItem.qml 2015-11-26 02:38:41 +0000 |
43 | @@ -17,7 +17,7 @@ |
44 | */ |
45 | |
46 | import QtQuick 2.4 |
47 | -import Ubuntu.Components 1.2 |
48 | +import Ubuntu.Components 1.3 |
49 | import Ubuntu.Components.ListItems 1.0 as ListItem |
50 | |
51 | /* |
52 | |
53 | === modified file 'app/components/FakeHeader.qml' |
54 | --- app/components/FakeHeader.qml 2015-10-18 15:48:28 +0000 |
55 | +++ app/components/FakeHeader.qml 2015-11-26 02:38:41 +0000 |
56 | @@ -17,22 +17,22 @@ |
57 | */ |
58 | |
59 | import QtQuick 2.4 |
60 | -import Ubuntu.Components 1.2 |
61 | +import Ubuntu.Components 1.3 |
62 | |
63 | Column { |
64 | id: fakeHeader |
65 | |
66 | - height: units.gu(9) |
67 | + height: units.gu(6.125) |
68 | |
69 | Rectangle { |
70 | - height: units.gu(7) |
71 | + height: units.gu(6) |
72 | width: parent.width |
73 | color: Theme.palette.normal.background |
74 | } |
75 | |
76 | Rectangle { |
77 | color: "#C9C9C9" |
78 | - height: units.gu(2) |
79 | + height: units.gu(0.125) |
80 | anchors.left: parent.left |
81 | anchors.right: parent.right |
82 | } |
83 | |
84 | === modified file 'app/components/FastScroll.qml' |
85 | --- app/components/FastScroll.qml 2015-06-18 01:42:03 +0000 |
86 | +++ app/components/FastScroll.qml 2015-11-26 02:38:41 +0000 |
87 | @@ -41,7 +41,7 @@ |
88 | |
89 | // FastScroll.qml |
90 | import QtQuick 2.4 |
91 | -import Ubuntu.Components 1.2 |
92 | +import Ubuntu.Components 1.3 |
93 | import "FastScroll.js" as Sections |
94 | |
95 | Item { |
96 | |
97 | === modified file 'app/components/ForecastDetailsDelegate.qml' |
98 | --- app/components/ForecastDetailsDelegate.qml 2015-10-09 18:35:21 +0000 |
99 | +++ app/components/ForecastDetailsDelegate.qml 2015-11-26 02:38:41 +0000 |
100 | @@ -17,7 +17,7 @@ |
101 | */ |
102 | |
103 | import QtQuick 2.4 |
104 | -import Ubuntu.Components 1.2 |
105 | +import Ubuntu.Components 1.3 |
106 | |
107 | Row { |
108 | height: icon.height |
109 | |
110 | === modified file 'app/components/HeaderRow.qml' |
111 | --- app/components/HeaderRow.qml 2015-07-27 21:41:09 +0000 |
112 | +++ app/components/HeaderRow.qml 2015-11-26 02:38:41 +0000 |
113 | @@ -18,7 +18,7 @@ |
114 | |
115 | import QtQuick 2.4 |
116 | import QtQuick.Layouts 1.1 |
117 | -import Ubuntu.Components 1.2 |
118 | +import Ubuntu.Components 1.3 |
119 | |
120 | RowLayout { |
121 | id: headerRow |
122 | |
123 | === modified file 'app/components/HomeGraphic.qml' |
124 | --- app/components/HomeGraphic.qml 2015-06-18 01:42:03 +0000 |
125 | +++ app/components/HomeGraphic.qml 2015-11-26 02:38:41 +0000 |
126 | @@ -17,7 +17,7 @@ |
127 | */ |
128 | |
129 | import QtQuick 2.4 |
130 | -import Ubuntu.Components 1.2 |
131 | +import Ubuntu.Components 1.3 |
132 | |
133 | Item { |
134 | height: units.gu(32) |
135 | |
136 | === modified file 'app/components/HomeHourly.qml' |
137 | --- app/components/HomeHourly.qml 2015-08-20 19:47:45 +0000 |
138 | +++ app/components/HomeHourly.qml 2015-11-26 02:38:41 +0000 |
139 | @@ -17,7 +17,7 @@ |
140 | */ |
141 | |
142 | import QtQuick 2.4 |
143 | -import Ubuntu.Components 1.2 |
144 | +import Ubuntu.Components 1.3 |
145 | |
146 | ListView { |
147 | id: homeHourly |
148 | |
149 | === modified file 'app/components/HomePageEmptyStateComponent.qml' |
150 | --- app/components/HomePageEmptyStateComponent.qml 2015-07-30 04:10:44 +0000 |
151 | +++ app/components/HomePageEmptyStateComponent.qml 2015-11-26 02:38:41 +0000 |
152 | @@ -17,7 +17,7 @@ |
153 | */ |
154 | |
155 | import QtQuick 2.4 |
156 | -import Ubuntu.Components 1.2 |
157 | +import Ubuntu.Components 1.3 |
158 | import "../components" |
159 | |
160 | |
161 | |
162 | === modified file 'app/components/HomeTempInfo.qml' |
163 | --- app/components/HomeTempInfo.qml 2015-10-16 00:14:28 +0000 |
164 | +++ app/components/HomeTempInfo.qml 2015-11-26 02:38:41 +0000 |
165 | @@ -17,7 +17,7 @@ |
166 | */ |
167 | |
168 | import QtQuick 2.4 |
169 | -import Ubuntu.Components 1.2 |
170 | +import Ubuntu.Components 1.3 |
171 | |
172 | Item { |
173 | id: homeTempInfoItem |
174 | |
175 | === modified file 'app/components/ListItemActions/CheckBox.qml' |
176 | --- app/components/ListItemActions/CheckBox.qml 2015-06-18 01:45:57 +0000 |
177 | +++ app/components/ListItemActions/CheckBox.qml 2015-11-26 02:38:41 +0000 |
178 | @@ -15,7 +15,7 @@ |
179 | */ |
180 | |
181 | import QtQuick 2.4 |
182 | -import Ubuntu.Components 1.2 |
183 | +import Ubuntu.Components 1.3 |
184 | |
185 | CheckBox { |
186 | checked: root.selected |
187 | |
188 | === modified file 'app/components/ListItemActions/Remove.qml' |
189 | --- app/components/ListItemActions/Remove.qml 2015-06-18 01:42:03 +0000 |
190 | +++ app/components/ListItemActions/Remove.qml 2015-11-26 02:38:41 +0000 |
191 | @@ -17,7 +17,7 @@ |
192 | */ |
193 | |
194 | import QtQuick 2.4 |
195 | -import Ubuntu.Components 1.2 |
196 | +import Ubuntu.Components 1.3 |
197 | |
198 | Action { |
199 | id: removeAction |
200 | |
201 | === modified file 'app/components/ListItemReorderComponent.qml' |
202 | --- app/components/ListItemReorderComponent.qml 2015-06-18 01:42:03 +0000 |
203 | +++ app/components/ListItemReorderComponent.qml 2015-11-26 02:38:41 +0000 |
204 | @@ -18,7 +18,7 @@ |
205 | */ |
206 | |
207 | import QtQuick 2.4 |
208 | -import Ubuntu.Components 1.2 |
209 | +import Ubuntu.Components 1.3 |
210 | |
211 | |
212 | Item { |
213 | |
214 | === modified file 'app/components/ListItemWithActions.qml' |
215 | --- app/components/ListItemWithActions.qml 2015-06-18 01:42:03 +0000 |
216 | +++ app/components/ListItemWithActions.qml 2015-11-26 02:38:41 +0000 |
217 | @@ -15,7 +15,7 @@ |
218 | */ |
219 | |
220 | import QtQuick 2.4 |
221 | -import Ubuntu.Components 1.2 |
222 | +import Ubuntu.Components 1.3 |
223 | import Ubuntu.Components.ListItems 1.0 as ListItem |
224 | |
225 | |
226 | |
227 | === modified file 'app/components/LoadingIndicator.qml' |
228 | --- app/components/LoadingIndicator.qml 2015-06-18 01:42:03 +0000 |
229 | +++ app/components/LoadingIndicator.qml 2015-11-26 02:38:41 +0000 |
230 | @@ -17,7 +17,7 @@ |
231 | */ |
232 | |
233 | import QtQuick 2.4 |
234 | -import Ubuntu.Components 1.2 |
235 | +import Ubuntu.Components 1.3 |
236 | |
237 | Rectangle { |
238 | id: indicator |
239 | |
240 | === modified file 'app/components/LocationsPageEmptyStateComponent.qml' |
241 | --- app/components/LocationsPageEmptyStateComponent.qml 2015-07-30 04:10:44 +0000 |
242 | +++ app/components/LocationsPageEmptyStateComponent.qml 2015-11-26 02:38:41 +0000 |
243 | @@ -17,7 +17,7 @@ |
244 | */ |
245 | |
246 | import QtQuick 2.4 |
247 | -import Ubuntu.Components 1.2 |
248 | +import Ubuntu.Components 1.3 |
249 | import "../components" |
250 | |
251 | |
252 | |
253 | === modified file 'app/components/MultiSelectHeadState.qml' |
254 | --- app/components/MultiSelectHeadState.qml 2015-06-18 01:42:03 +0000 |
255 | +++ app/components/MultiSelectHeadState.qml 2015-11-26 02:38:41 +0000 |
256 | @@ -17,7 +17,7 @@ |
257 | */ |
258 | |
259 | import QtQuick 2.4 |
260 | -import Ubuntu.Components 1.2 |
261 | +import Ubuntu.Components 1.3 |
262 | |
263 | PageHeadState { |
264 | id: selectionState |
265 | |
266 | === modified file 'app/components/MultiSelectListView.qml' |
267 | --- app/components/MultiSelectListView.qml 2015-06-18 01:42:03 +0000 |
268 | +++ app/components/MultiSelectListView.qml 2015-11-26 02:38:41 +0000 |
269 | @@ -18,7 +18,7 @@ |
270 | */ |
271 | |
272 | import QtQuick 2.4 |
273 | -import Ubuntu.Components 1.2 |
274 | +import Ubuntu.Components 1.3 |
275 | |
276 | |
277 | WeatherListView { |
278 | |
279 | === modified file 'app/components/NetworkErrorStateComponent.qml' |
280 | --- app/components/NetworkErrorStateComponent.qml 2015-10-19 03:05:53 +0000 |
281 | +++ app/components/NetworkErrorStateComponent.qml 2015-11-26 02:38:41 +0000 |
282 | @@ -17,7 +17,7 @@ |
283 | */ |
284 | |
285 | import QtQuick 2.4 |
286 | -import Ubuntu.Components 1.2 |
287 | +import Ubuntu.Components 1.3 |
288 | import "../components" |
289 | |
290 | |
291 | |
292 | === modified file 'app/components/NoAPIKeyErrorStateComponent.qml' |
293 | --- app/components/NoAPIKeyErrorStateComponent.qml 2015-11-10 02:02:19 +0000 |
294 | +++ app/components/NoAPIKeyErrorStateComponent.qml 2015-11-26 02:38:41 +0000 |
295 | @@ -17,7 +17,7 @@ |
296 | */ |
297 | |
298 | import QtQuick 2.4 |
299 | -import Ubuntu.Components 1.2 |
300 | +import Ubuntu.Components 1.5 |
301 | import "../components" |
302 | |
303 | |
304 | |
305 | === modified file 'app/components/PageWithBottomEdge.qml' |
306 | --- app/components/PageWithBottomEdge.qml 2015-10-18 15:48:28 +0000 |
307 | +++ app/components/PageWithBottomEdge.qml 2015-11-26 02:38:41 +0000 |
308 | @@ -63,7 +63,7 @@ |
309 | */ |
310 | |
311 | import QtQuick 2.4 |
312 | -import Ubuntu.Components 1.2 |
313 | +import Ubuntu.Components 1.3 |
314 | |
315 | Page { |
316 | id: page |
317 | @@ -77,7 +77,8 @@ |
318 | property bool reloadBottomEdgePage: true |
319 | |
320 | readonly property alias bottomEdgePage: edgeLoader.item |
321 | - readonly property bool isReady: ((bottomEdge.y === fakeHeader.height) && bottomEdgePageLoaded && edgeLoader.item.active) // CUSTOM change to flag to allow for FakeHeader height |
322 | + // FIXME: 6.125 is the header.height |
323 | + readonly property bool isReady: ((bottomEdge.y === units.gu(6.125)) && bottomEdgePageLoaded && edgeLoader.item.active) // CUSTOM change to flag to allow for FakeHeader height |
324 | readonly property bool isCollapsed: (bottomEdge.y === page.height) |
325 | readonly property bool bottomEdgePageLoaded: (edgeLoader.status == Loader.Ready) |
326 | |
327 | @@ -85,7 +86,7 @@ |
328 | property int _areaWhenExpanded: 0 |
329 | |
330 | // CUSTOM properties to allow changing of color |
331 | - property alias tipColor: tip.color |
332 | + property alias tipColor: tip.backgroundColor // CUSTOM color => backgroundColor |
333 | property alias tipLabelColor: tipLabel.color |
334 | |
335 | signal bottomEdgeReleased() |
336 | @@ -108,7 +109,7 @@ |
337 | edgeLoader.item.active = true |
338 | page.pageStack.push(edgeLoader.item) |
339 | if (edgeLoader.item.flickable) { |
340 | - edgeLoader.item.flickable.contentY = -page.header.height |
341 | + edgeLoader.item.flickable.contentY = -units.gu(6.125) // FIXME: 6.125 is the header.height |
342 | edgeLoader.item.flickable.returnToBounds() |
343 | } |
344 | if (edgeLoader.item.ready) |
345 | @@ -180,7 +181,7 @@ |
346 | z: 1 |
347 | width: tipLabel.paintedWidth + units.gu(6) |
348 | height: bottomEdge.tipHeight + units.gu(1) |
349 | - color: Theme.palette.normal.overlay |
350 | + backgroundColor: Theme.palette.normal.overlay // CUSTOM color => backgroundColor |
351 | Label { |
352 | id: tipLabel |
353 | |
354 | @@ -278,7 +279,8 @@ |
355 | left: parent.left |
356 | right: parent.right |
357 | } |
358 | - y: -fakeHeader.height + (fakeHeader.height * (page.height - bottomEdge.y)) / (page.height - fakeHeader.height) |
359 | + // FIXME: 6.125 is the header.height |
360 | + y: -units.gu(6.125) + (units.gu(6.125) * (page.height - bottomEdge.y)) / (page.height - units.gu(6.125)) |
361 | z: bgVisual.z + 1 |
362 | |
363 | Behavior on y { |
364 | @@ -294,7 +296,7 @@ |
365 | |
366 | readonly property int tipHeight: units.gu(3) |
367 | // CUSTOM value |
368 | - readonly property int pageStartY: fakeHeader.height |
369 | + readonly property int pageStartY: units.gu(6.125) // FIXME: 6.125 is the header.height |
370 | |
371 | z: 1 |
372 | color: Theme.palette.normal.background |
373 | @@ -318,7 +320,7 @@ |
374 | // CUSTOM |
375 | PropertyChanges { |
376 | target: fakeHeader |
377 | - y: -fakeHeader.height |
378 | + y: -units.gu(6.125) // FIXME: 6.125 is the header.height |
379 | } |
380 | }, |
381 | State { |
382 | @@ -366,7 +368,8 @@ |
383 | SmoothedAnimation { |
384 | target: edgeLoader |
385 | property: "anchors.topMargin" |
386 | - to: - units.gu(4) |
387 | + // CUSTOM make small to reduce 'bump' when pushing |
388 | + to: -0.1 // FIXME: 6.125 is header height |
389 | duration: UbuntuAnimation.FastDuration |
390 | easing.type: Easing.Linear |
391 | } |
392 | @@ -418,6 +421,9 @@ |
393 | tip.forceActiveFocus() |
394 | } |
395 | |
396 | + // CUSTOM reset the topMargin for next pull |
397 | + edgeLoader.anchors.topMargin = units.gu(6.125) |
398 | + |
399 | // notify |
400 | page.bottomEdgeDismissed() |
401 | |
402 | @@ -441,14 +447,19 @@ |
403 | id: edgeLoader |
404 | |
405 | asynchronous: true |
406 | - anchors.fill: parent |
407 | + anchors { |
408 | + fill: parent |
409 | + topMargin: units.gu(6.125) // CUSTOM |
410 | + } |
411 | //WORKAROUND: The SDK move the page contents down to allocate space for the header we need to avoid that during the page dragging |
412 | + /* // CUSTOM |
413 | Binding { |
414 | target: edgeLoader.status === Loader.Ready ? edgeLoader : null |
415 | property: "anchors.topMargin" |
416 | value: edgeLoader.item && edgeLoader.item.flickable ? edgeLoader.item.flickable.contentY : 0 |
417 | when: !page.isReady |
418 | } |
419 | + */ |
420 | |
421 | onLoaded: { |
422 | tip.forceActiveFocus() |
423 | |
424 | === modified file 'app/components/SettingsButton.qml' |
425 | --- app/components/SettingsButton.qml 2015-09-05 00:03:33 +0000 |
426 | +++ app/components/SettingsButton.qml 2015-11-26 02:38:41 +0000 |
427 | @@ -17,7 +17,7 @@ |
428 | */ |
429 | |
430 | import QtQuick 2.4 |
431 | -import Ubuntu.Components 1.2 |
432 | +import Ubuntu.Components 1.3 |
433 | |
434 | |
435 | AbstractButton { |
436 | |
437 | === modified file 'app/components/StandardListItem.qml' |
438 | --- app/components/StandardListItem.qml 2015-06-21 14:54:05 +0000 |
439 | +++ app/components/StandardListItem.qml 2015-11-26 02:38:41 +0000 |
440 | @@ -18,7 +18,7 @@ |
441 | |
442 | import QtQuick 2.4 |
443 | import QtQuick.Layouts 1.1 |
444 | -import Ubuntu.Components 1.2 |
445 | +import Ubuntu.Components 1.3 |
446 | |
447 | ListItem { |
448 | id: listItem |
449 | |
450 | === modified file 'app/components/WeatherListItem.qml' |
451 | --- app/components/WeatherListItem.qml 2015-09-29 16:09:21 +0000 |
452 | +++ app/components/WeatherListItem.qml 2015-11-26 02:38:41 +0000 |
453 | @@ -18,7 +18,7 @@ |
454 | */ |
455 | |
456 | import QtQuick 2.4 |
457 | -import Ubuntu.Components 1.2 |
458 | +import Ubuntu.Components 1.3 |
459 | |
460 | ListItemWithActions { |
461 | id: root |
462 | |
463 | === modified file 'app/components/WeatherListView.qml' |
464 | --- app/components/WeatherListView.qml 2015-06-18 01:42:03 +0000 |
465 | +++ app/components/WeatherListView.qml 2015-11-26 02:38:41 +0000 |
466 | @@ -18,7 +18,7 @@ |
467 | */ |
468 | |
469 | import QtQuick 2.4 |
470 | -import Ubuntu.Components 1.2 |
471 | +import Ubuntu.Components 1.3 |
472 | |
473 | |
474 | ListView { |
475 | |
476 | === modified file 'app/ubuntu-weather-app.qml' |
477 | --- app/ubuntu-weather-app.qml 2015-10-14 23:31:23 +0000 |
478 | +++ app/ubuntu-weather-app.qml 2015-11-26 02:38:41 +0000 |
479 | @@ -18,7 +18,7 @@ |
480 | |
481 | import QtQuick 2.4 |
482 | import Qt.labs.settings 1.0 |
483 | -import Ubuntu.Components 1.2 |
484 | +import Ubuntu.Components 1.3 |
485 | import "components" |
486 | import "data" as Data |
487 | import "data/WeatherApi.js" as WeatherApi |
488 | |
489 | === modified file 'app/ui/AddLocationPage.qml' |
490 | --- app/ui/AddLocationPage.qml 2015-09-02 23:36:17 +0000 |
491 | +++ app/ui/AddLocationPage.qml 2015-11-26 02:38:41 +0000 |
492 | @@ -17,8 +17,8 @@ |
493 | */ |
494 | |
495 | import QtQuick 2.4 |
496 | -import Ubuntu.Components 1.2 |
497 | -import Ubuntu.Components.Popups 1.0 |
498 | +import Ubuntu.Components 1.3 |
499 | +import Ubuntu.Components.Popups 1.3 |
500 | import "../components" |
501 | import "../data/CitiesList.js" as Cities |
502 | import "../data/WeatherApi.js" as WeatherApi |
503 | @@ -180,13 +180,15 @@ |
504 | |
505 | ListView { |
506 | id: locationList |
507 | + anchors { |
508 | + fill: parent |
509 | + rightMargin: fastScroll.showing ? fastScroll.width - units.gu(1) : 0 |
510 | + topMargin: units.gu(2) |
511 | + } |
512 | + |
513 | objectName: "locationList" |
514 | - |
515 | clip: true |
516 | currentIndex: -1 |
517 | - anchors.fill: parent |
518 | - anchors.rightMargin: fastScroll.showing ? fastScroll.width - units.gu(1) |
519 | - : 0 |
520 | |
521 | function getSectionText(index) { |
522 | return citiesModel.get(index).name.substring(0,1) |
523 | |
524 | === modified file 'app/ui/HomePage.qml' |
525 | --- app/ui/HomePage.qml 2015-11-10 02:13:04 +0000 |
526 | +++ app/ui/HomePage.qml 2015-11-26 02:38:41 +0000 |
527 | @@ -17,7 +17,7 @@ |
528 | */ |
529 | |
530 | import QtQuick 2.4 |
531 | -import Ubuntu.Components 1.2 |
532 | +import Ubuntu.Components 1.3 |
533 | import "../components" |
534 | import "../data/keys.js" as Keys |
535 | |
536 | @@ -84,6 +84,12 @@ |
537 | return new Date(dateData.year, dateData.month, dateData.date, dateData.hours, dateData.minutes) |
538 | } |
539 | |
540 | + // Do not show the Page Header |
541 | + head { |
542 | + locked: true |
543 | + visible: false |
544 | + } |
545 | + |
546 | /* |
547 | Background for the PageWithBottomEdge |
548 | */ |
549 | |
550 | === modified file 'app/ui/LocationPane.qml' |
551 | --- app/ui/LocationPane.qml 2015-10-22 15:17:02 +0000 |
552 | +++ app/ui/LocationPane.qml 2015-11-26 02:38:41 +0000 |
553 | @@ -17,7 +17,7 @@ |
554 | */ |
555 | |
556 | import QtQuick 2.4 |
557 | -import Ubuntu.Components 1.2 |
558 | +import Ubuntu.Components 1.3 |
559 | import Ubuntu.Components.ListItems 0.1 as ListItem |
560 | import "../components" |
561 | import "../data/suncalc.js" as SunCalc |
562 | |
563 | === modified file 'app/ui/LocationsPage.qml' |
564 | --- app/ui/LocationsPage.qml 2015-10-22 12:49:35 +0000 |
565 | +++ app/ui/LocationsPage.qml 2015-11-26 02:38:41 +0000 |
566 | @@ -17,7 +17,7 @@ |
567 | */ |
568 | |
569 | import QtQuick 2.4 |
570 | -import Ubuntu.Components 1.2 |
571 | +import Ubuntu.Components 1.3 |
572 | import Ubuntu.Components.ListItems 0.1 as ListItem |
573 | import "../components" |
574 | import "../components/ListItemActions" |
575 | @@ -66,7 +66,7 @@ |
576 | anchors { |
577 | fill: parent |
578 | // TODO: Fix this offset |
579 | - topMargin: locationsPage.header ? -locationsPage.header.height : 0 |
580 | + topMargin: -units.gu(6.125) // FIXME: 6.125 is the header.height |
581 | } |
582 | model: ListModel { |
583 | id: locationsModel |
584 | |
585 | === modified file 'app/ui/SettingsPage.qml' |
586 | --- app/ui/SettingsPage.qml 2015-08-10 08:13:31 +0000 |
587 | +++ app/ui/SettingsPage.qml 2015-11-26 02:38:41 +0000 |
588 | @@ -17,7 +17,7 @@ |
589 | */ |
590 | |
591 | import QtQuick 2.4 |
592 | -import Ubuntu.Components 1.2 |
593 | +import Ubuntu.Components 1.3 |
594 | import "../components" |
595 | |
596 | Page { |
597 | |
598 | === modified file 'app/ui/settings/DataProviderPage.qml' |
599 | --- app/ui/settings/DataProviderPage.qml 2015-10-14 23:31:23 +0000 |
600 | +++ app/ui/settings/DataProviderPage.qml 2015-11-26 02:38:41 +0000 |
601 | @@ -17,7 +17,7 @@ |
602 | */ |
603 | |
604 | import QtQuick 2.4 |
605 | -import Ubuntu.Components 1.2 |
606 | +import Ubuntu.Components 1.3 |
607 | import "../../components" |
608 | import "../../data/keys.js" as Keys |
609 | |
610 | |
611 | === modified file 'app/ui/settings/LocationPage.qml' |
612 | --- app/ui/settings/LocationPage.qml 2015-07-27 00:57:57 +0000 |
613 | +++ app/ui/settings/LocationPage.qml 2015-11-26 02:38:41 +0000 |
614 | @@ -17,7 +17,7 @@ |
615 | */ |
616 | |
617 | import QtQuick 2.4 |
618 | -import Ubuntu.Components 1.2 |
619 | +import Ubuntu.Components 1.3 |
620 | import Ubuntu.Components.ListItems 1.0 as ListItem |
621 | |
622 | Page { |
623 | |
624 | === modified file 'app/ui/settings/RefreshIntervalPage.qml' |
625 | --- app/ui/settings/RefreshIntervalPage.qml 2015-06-21 14:54:05 +0000 |
626 | +++ app/ui/settings/RefreshIntervalPage.qml 2015-11-26 02:38:41 +0000 |
627 | @@ -17,7 +17,7 @@ |
628 | */ |
629 | |
630 | import QtQuick 2.4 |
631 | -import Ubuntu.Components 1.2 |
632 | +import Ubuntu.Components 1.3 |
633 | import "../../components" |
634 | |
635 | Page { |
636 | |
637 | === modified file 'app/ui/settings/UnitsPage.qml' |
638 | --- app/ui/settings/UnitsPage.qml 2015-08-10 16:27:31 +0000 |
639 | +++ app/ui/settings/UnitsPage.qml 2015-11-26 02:38:41 +0000 |
640 | @@ -17,7 +17,7 @@ |
641 | */ |
642 | |
643 | import QtQuick 2.4 |
644 | -import Ubuntu.Components 1.2 |
645 | +import Ubuntu.Components 1.3 |
646 | import "../../components" |
647 | |
648 | Page { |
649 | |
650 | === modified file 'debian/changelog' |
651 | --- debian/changelog 2015-11-10 02:02:19 +0000 |
652 | +++ debian/changelog 2015-11-26 02:38:41 +0000 |
653 | @@ -2,6 +2,7 @@ |
654 | |
655 | [ Andrew Hayzen ] |
656 | * Bump version to 3.1 |
657 | + * Update app to use the Ubuntu Components version 1.3 (LP: #1508363) |
658 | |
659 | [ Bartosz Kosiorek ] |
660 | * Change message text according to Ubuntu UX comment (LP: #1493466) |
661 | |
662 | === modified file 'manifest.json.in' |
663 | --- manifest.json.in 2015-10-24 19:24:48 +0000 |
664 | +++ manifest.json.in 2015-11-26 02:38:41 +0000 |
665 | @@ -1,7 +1,7 @@ |
666 | { |
667 | "architecture": "all", |
668 | "description": "A weather forecast application for Ubuntu with support for multiple online weather data sources", |
669 | - "framework": "ubuntu-sdk-15.04-qml", |
670 | + "framework": "ubuntu-sdk-15.04.1-qml", |
671 | "hooks": { |
672 | "weather": { |
673 | "apparmor": "ubuntu-weather-app.apparmor", |
674 | |
675 | === modified file 'po/com.ubuntu.weather.pot' |
676 | --- po/com.ubuntu.weather.pot 2015-11-10 02:02:19 +0000 |
677 | +++ po/com.ubuntu.weather.pot 2015-11-26 02:38:41 +0000 |
678 | @@ -8,7 +8,7 @@ |
679 | msgstr "" |
680 | "Project-Id-Version: ubuntu-weather-app\n" |
681 | "Report-Msgid-Bugs-To: \n" |
682 | -"POT-Creation-Date: 2015-11-09 19:59-0600\n" |
683 | +"POT-Creation-Date: 2015-11-26 02:04+0000\n" |
684 | "PO-Revision-Date: YEAR-MO-DA HO:MI+ZONE\n" |
685 | "Last-Translator: FULL NAME <EMAIL@ADDRESS>\n" |
686 | "Language-Team: LANGUAGE <LL@li.org>\n" |
687 | @@ -120,23 +120,23 @@ |
688 | msgid "Search city" |
689 | msgstr "" |
690 | |
691 | -#: ../app/ui/AddLocationPage.qml:297 |
692 | +#: ../app/ui/AddLocationPage.qml:299 |
693 | msgid "No city found" |
694 | msgstr "" |
695 | |
696 | -#: ../app/ui/AddLocationPage.qml:310 |
697 | +#: ../app/ui/AddLocationPage.qml:312 |
698 | msgid "Couldn't load weather data, please try later again!" |
699 | msgstr "" |
700 | |
701 | -#: ../app/ui/AddLocationPage.qml:320 |
702 | +#: ../app/ui/AddLocationPage.qml:322 |
703 | msgid "Location already added." |
704 | msgstr "" |
705 | |
706 | -#: ../app/ui/AddLocationPage.qml:323 |
707 | +#: ../app/ui/AddLocationPage.qml:325 |
708 | msgid "OK" |
709 | msgstr "" |
710 | |
711 | -#: ../app/ui/HomePage.qml:31 ../app/ui/LocationsPage.qml:29 |
712 | +#: ../app/ui/HomePage.qml:32 ../app/ui/LocationsPage.qml:29 |
713 | msgid "Locations" |
714 | msgstr "" |
715 | |
716 | |
717 | === modified file 'tests/autopilot/ubuntu_weather_app/__init__.py' |
718 | --- tests/autopilot/ubuntu_weather_app/__init__.py 2015-10-22 12:49:35 +0000 |
719 | +++ tests/autopilot/ubuntu_weather_app/__init__.py 2015-11-26 02:38:41 +0000 |
720 | @@ -85,12 +85,17 @@ |
721 | self.bottomEdgePageLoaded.wait_for(True) |
722 | |
723 | try: |
724 | - action_item = self.wait_select_single(objectName='bottomEdgeTip') |
725 | + action_item = self.wait_select_single("UCUbuntuShape", |
726 | + objectName='bottomEdgeTip') |
727 | action_item.visible.wait_for(True) |
728 | + |
729 | + # Pick the middle horizontally and vertically of the bottomEdgeTip |
730 | start_x = (action_item.globalRect.x + |
731 | (action_item.globalRect.width * 0.5)) |
732 | - start_y = action_item.globalRect.y |
733 | + start_y = (action_item.globalRect.y + |
734 | + (action_item.globalRect.height * 0.5)) |
735 | stop_y = start_y - (self.height * 0.7) |
736 | + |
737 | self.pointing_device.drag(start_x, start_y, |
738 | start_x, stop_y, rate=2) |
739 | self.isReady.wait_for(True) |
740 | @@ -130,7 +135,7 @@ |
741 | return header.select_single("TextField", objectName="searchField") |
742 | |
743 | def is_empty_label_visible(self): |
744 | - return self.select_single("Label", objectName="noCity").visible |
745 | + return self.select_single("UCLabel", objectName="noCity").visible |
746 | |
747 | def search(self, value): |
748 | self.click_search_action() |
749 | @@ -226,7 +231,7 @@ |
750 | |
751 | def get_settings_button(self): |
752 | return self.select_single( |
753 | - "AbstractButton", objectName="settingsButton") |
754 | + "UCAbstractButton", objectName="settingsButton") |
755 | |
756 | |
757 | class LocationsPage(Page): |
758 | @@ -308,7 +313,7 @@ |
759 | |
760 | class WeatherListItem(UbuntuUIToolkitCustomProxyObjectBase): |
761 | def get_name(self): |
762 | - return self.select_single("Label", objectName="name").text |
763 | + return self.select_single("UCLabel", objectName="name").text |
764 | |
765 | @click_object |
766 | def select_remove(self): |
767 | |
768 | === modified file 'ubuntu-weather-app.desktop.in.in' |
769 | --- ubuntu-weather-app.desktop.in.in 2015-10-24 19:35:02 +0000 |
770 | +++ ubuntu-weather-app.desktop.in.in 2015-11-26 02:38:41 +0000 |
771 | @@ -10,4 +10,5 @@ |
772 | X-Ubuntu-Touch=true |
773 | X-Ubuntu-Default-Department-ID=accessories |
774 | X-Ubuntu-Splash-Color=#F5F5F5 |
775 | +X-Ubuntu-Splash-Show-Header=true |
776 | X-Ubuntu-Supported-Orientations=portrait |
FAILED: Continuous integration, rev:154 91.189. 93.70:8080/ job/ubuntu- weather- app-reboot- ci/289/ 91.189. 93.70:8080/ job/ubuntu- weather- app-reboot- vivid-amd64- ci/289/ console
http://
Executed test runs:
FAILURE: http://
Click here to trigger a rebuild: 91.189. 93.70:8080/ job/ubuntu- weather- app-reboot- ci/289/ rebuild
http://