Merge lp:~rpadovani/ubuntu-calculator-app/bumpToolkit into lp:ubuntu-calculator-app
- bumpToolkit
- Merge into trunk
Status: | Merged | ||||||||
---|---|---|---|---|---|---|---|---|---|
Approved by: | Riccardo Padovani | ||||||||
Approved revision: | 242 | ||||||||
Merged at revision: | 241 | ||||||||
Proposed branch: | lp:~rpadovani/ubuntu-calculator-app/bumpToolkit | ||||||||
Merge into: | lp:ubuntu-calculator-app | ||||||||
Diff against target: |
539 lines (+148/-100) 16 files modified
app/engine/CalculationHistory.qml (+2/-2) app/ubuntu-calculator-app.qml (+61/-64) app/ui/CalcKeyboard.qml (+3/-3) app/ui/FavouritePage.qml (+3/-3) app/ui/KeyboardButton.qml (+3/-3) app/ui/KeyboardPage.qml (+19/-2) app/ui/LandscapeKeyboard.qml (+20/-2) app/ui/PortraitKeyboard.qml (+19/-2) app/ui/Screen.qml (+2/-2) app/ui/ScrollableView.qml (+2/-2) app/upstreamcomponents/EmptyState.qml (+2/-2) app/upstreamcomponents/ListItemWithActions.qml (+3/-3) app/upstreamcomponents/ListItemWithActionsCheckBox.qml (+3/-3) app/upstreamcomponents/MultipleSelectionVisualModel.qml (+1/-1) app/upstreamcomponents/PageWithBottomEdge.qml (+4/-5) manifest.json.in (+1/-1) |
||||||||
To merge this branch: | bzr merge lp:~rpadovani/ubuntu-calculator-app/bumpToolkit | ||||||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Jenkins Bot | continuous-integration | Approve | |
Bartosz Kosiorek | Needs Fixing | ||
Ubuntu Phone Apps Jenkins Bot | continuous-integration | Approve | |
Review via email: mp+277012@code.launchpad.net |
Commit message
Update QtQuick and ubuntu-ui-toolkit
Description of the change
- Update all QtQuick to 2.4 and toolkit to 1.3
- Drop custom header in favour of standard one
- Fix bug #1413789
Jenkins Bot (ubuntu-core-apps-jenkins-bot) wrote : | # |
Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) wrote : | # |
PASSED: Continuous integration, rev:235
http://
Executed test runs:
SUCCESS: http://
Click here to trigger a rebuild:
http://
Bartosz Kosiorek (gang65) wrote : | # |
Some small inline comment.
I will intense test it later today.
Jenkins Bot (ubuntu-core-apps-jenkins-bot) wrote : | # |
PASSED: Continuous integration, rev:236
https:/
Executed test runs:
None: https:/
Click here to trigger a rebuild:
https:/
Riccardo Padovani (rpadovani) wrote : | # |
Thanks!
Jenkins Bot (ubuntu-core-apps-jenkins-bot) wrote : | # |
PASSED: Continuous integration, rev:237
https:/
Executed test runs:
None: https:/
Click here to trigger a rebuild:
https:/
Bartosz Kosiorek (gang65) wrote : | # |
Thanks Riccardo.
I noticed that following warnings are appearing:
file://
and
uses confusion which is the active page that sets the title and actions.
file://
file://
file://
file://
and error during scrolling selected entries:
file://
Could you please try to fix that?
There are also errors:
file://
Bartosz Kosiorek (gang65) wrote : | # |
BTW The windows title is working perfectly now! No more "com.ubuntu.
One more small issue:
When you run first time Calculator, the virtual keyboard is slighly shifted to bottom.
After first click, it moves in correct place.
Could you please take a look at this.
Jenkins Bot (ubuntu-core-apps-jenkins-bot) wrote : | # |
PASSED: Continuous integration, rev:240
https:/
Executed test runs:
None: https:/
Click here to trigger a rebuild:
https:/
Riccardo Padovani (rpadovani) wrote : | # |
I fixed all the warnings, I'm still investigating about the virtual keyboard position
Riccardo Padovani (rpadovani) wrote : | # |
I'm quite puzzled about the bug, it happens randomly, not all the time, probably 1 on 3. I have no idea of which is the root case, and I tried a lot of different things.
I'm sure we've already seen something like this, but I don't remember how we fixed it.
If you have any suggestions, please let me know!
Riccardo Padovani (rpadovani) wrote : | # |
So, some progress on this.
It seems that when the app is loaded the pagestack changes its height a lot of times:
if you add onHeightChanged: console.
qml: pagestack 565.7091325559794
qml: pagestack 560
qml: pagestack 565.3556258961664
qml: pagestack 560
qml: pagestack 565.0021192363535
qml: pagestack 560
qml: pagestack 564.6486125765405
qml: pagestack 560
qml: pagestack 564.2951059167276
qml: pagestack 560
qml: pagestack 563.9415992569145
qml: pagestack 560
qml: pagestack 563.5880925971016
qml: pagestack 560
qml: pagestack 563.2345859372887
qml: pagestack 560
qml: pagestack 562.8810792774757
qml: pagestack 560
qml: pagestack 562.5275726176627
qml: pagestack 560
qml: pagestack 562.1740659578497
qml: pagestack 560
qml: pagestack 561.8205592980368
qml: pagestack 560
qml: pagestack 561.4670526382238
qml: pagestack 560
qml: pagestack 561.1135459784108
qml: pagestack 560
qml: pagestack 560.7600393185979
qml: pagestack 560
qml: pagestack 560.4065326587848
qml: pagestack 560
qml: pagestack 560.0746782818856
qml: pagestack 560
I need to understand what's the root cause of this change - when I'll be able to isolate it, I think I'll be able to fix it
Riccardo Padovani (rpadovani) wrote : | # |
Investigating more, it is related to the header. If I don't hide the header it doesn't happen.
I think is related to https:/
Jenkins Bot (ubuntu-core-apps-jenkins-bot) wrote : | # |
PASSED: Continuous integration, rev:241
https:/
Executed test runs:
None: https:/
Click here to trigger a rebuild:
https:/
Riccardo Padovani (rpadovani) wrote : | # |
Okay, I found a workaround until the bug is fixed upstream.
It introduces a bug in the desktop mode (when you resize the window, the app scrolls down) but I think is way less important that this one
Bartosz Kosiorek (gang65) wrote : | # |
Hello Riccardo.
Could you please submit bug report for that issue?
It will be great to resolve it upstream.
Jenkins Bot (ubuntu-core-apps-jenkins-bot) wrote : | # |
FAILED: Autolanding.
More details in the following jenkins job:
https:/
Executed test runs:
None: https:/
Jenkins Bot (ubuntu-core-apps-jenkins-bot) wrote : | # |
FAILED: Autolanding.
More details in the following jenkins job:
https:/
Executed test runs:
None: https:/
Bartosz Kosiorek (gang65) wrote : | # |
There are same conflicts. Please resolve them. After that just top approve that.
- 242. By Riccardo Padovani
-
Merge upstream and fix conflicts
Jenkins Bot (ubuntu-core-apps-jenkins-bot) wrote : | # |
PASSED: Continuous integration, rev:242
https:/
Executed test runs:
None: https:/
Click here to trigger a rebuild:
https:/
Riccardo Padovani (rpadovani) wrote : | # |
Thanks Bartosz, I fixed conflicts.
The bug upstream is https:/
Preview Diff
1 | === modified file 'app/engine/CalculationHistory.qml' | |||
2 | --- app/engine/CalculationHistory.qml 2015-03-13 14:51:17 +0000 | |||
3 | +++ app/engine/CalculationHistory.qml 2015-11-24 13:43:45 +0000 | |||
4 | @@ -1,5 +1,5 @@ | |||
5 | 1 | /* | 1 | /* |
7 | 2 | * Copyright (C) 2014 Canonical, Ltd. | 2 | * Copyright (C) 2014-2015 Canonical, Ltd. |
8 | 3 | * | 3 | * |
9 | 4 | * This program is free software; you can redistribute it and/or modify | 4 | * This program is free software; you can redistribute it and/or modify |
10 | 5 | * it under the terms of the GNU General Public License as published by | 5 | * it under the terms of the GNU General Public License as published by |
11 | @@ -13,7 +13,7 @@ | |||
12 | 13 | * You should have received a copy of the GNU General Public License | 13 | * You should have received a copy of the GNU General Public License |
13 | 14 | * along with this program. If not, see <http://www.gnu.org/licenses/>. | 14 | * along with this program. If not, see <http://www.gnu.org/licenses/>. |
14 | 15 | */ | 15 | */ |
16 | 16 | import QtQuick 2.3 | 16 | import QtQuick 2.4 |
17 | 17 | import QtQuick.LocalStorage 2.0 | 17 | import QtQuick.LocalStorage 2.0 |
18 | 18 | 18 | ||
19 | 19 | Item { | 19 | Item { |
20 | 20 | 20 | ||
21 | === modified file 'app/ubuntu-calculator-app.qml' | |||
22 | --- app/ubuntu-calculator-app.qml 2015-11-22 13:59:30 +0000 | |||
23 | +++ app/ubuntu-calculator-app.qml 2015-11-24 13:43:45 +0000 | |||
24 | @@ -1,5 +1,5 @@ | |||
25 | 1 | /* | 1 | /* |
27 | 2 | * Copyright (C) 2014 Canonical Ltd | 2 | * Copyright (C) 2014-2015 Canonical Ltd |
28 | 3 | * | 3 | * |
29 | 4 | * This file is part of Ubuntu Calculator App | 4 | * This file is part of Ubuntu Calculator App |
30 | 5 | * | 5 | * |
31 | @@ -15,9 +15,9 @@ | |||
32 | 15 | * You should have received a copy of the GNU General Public License | 15 | * You should have received a copy of the GNU General Public License |
33 | 16 | * along with this program. If not, see <http://www.gnu.org/licenses/>. | 16 | * along with this program. If not, see <http://www.gnu.org/licenses/>. |
34 | 17 | */ | 17 | */ |
38 | 18 | import QtQuick 2.3 | 18 | import QtQuick 2.4 |
39 | 19 | import Ubuntu.Components 1.1 | 19 | import Ubuntu.Components 1.3 |
40 | 20 | import Ubuntu.Components.Themes.Ambiance 0.1 | 20 | import Ubuntu.Components.Themes.Ambiance 1.3 |
41 | 21 | 21 | ||
42 | 22 | import "ui" | 22 | import "ui" |
43 | 23 | import "upstreamcomponents" | 23 | import "upstreamcomponents" |
44 | @@ -31,8 +31,6 @@ | |||
45 | 31 | objectName: "calculator"; | 31 | objectName: "calculator"; |
46 | 32 | applicationName: "com.ubuntu.calculator"; | 32 | applicationName: "com.ubuntu.calculator"; |
47 | 33 | 33 | ||
48 | 34 | // Removes the old toolbar and enables new features of the new header. | ||
49 | 35 | useDeprecatedToolbar: false; | ||
50 | 36 | automaticOrientation: true | 34 | automaticOrientation: true |
51 | 37 | anchorToKeyboard: textInputField.visible ? false : true | 35 | anchorToKeyboard: textInputField.visible ? false : true |
52 | 38 | 36 | ||
53 | @@ -109,6 +107,7 @@ | |||
54 | 109 | 107 | ||
55 | 110 | // Maximum length of the result number | 108 | // Maximum length of the result number |
56 | 111 | var NUMBER_LENGTH_LIMIT = 14; | 109 | var NUMBER_LENGTH_LIMIT = 14; |
57 | 110 | |||
58 | 112 | if (bigNumberToFormat.toString().length > NUMBER_LENGTH_LIMIT) { | 111 | if (bigNumberToFormat.toString().length > NUMBER_LENGTH_LIMIT) { |
59 | 113 | var resultLength = mathJs.format(bigNumberToFormat, {exponential: {lower: 1e-10, upper: 1e10}, | 112 | var resultLength = mathJs.format(bigNumberToFormat, {exponential: {lower: 1e-10, upper: 1e10}, |
60 | 114 | precision: NUMBER_LENGTH_LIMIT}).toString().length; | 113 | precision: NUMBER_LENGTH_LIMIT}).toString().length; |
61 | @@ -238,14 +237,19 @@ | |||
62 | 238 | id: mainStack | 237 | id: mainStack |
63 | 239 | 238 | ||
64 | 240 | Component.onCompleted: { | 239 | Component.onCompleted: { |
65 | 241 | |||
66 | 242 | push(calculatorPage); | 240 | push(calculatorPage); |
67 | 243 | calculatorPage.forceActiveFocus(); | 241 | calculatorPage.forceActiveFocus(); |
68 | 244 | } | 242 | } |
69 | 245 | 243 | ||
70 | 244 | onHeightChanged: scrollableView.scrollToBottom(); | ||
71 | 245 | anchors.fill: parent | ||
72 | 246 | |||
73 | 246 | PageWithBottomEdge { | 247 | PageWithBottomEdge { |
74 | 247 | id: calculatorPage | 248 | id: calculatorPage |
75 | 248 | 249 | ||
76 | 250 | title: i18n.tr("Calculator") | ||
77 | 251 | anchors.fill: parent | ||
78 | 252 | |||
79 | 249 | bottomEdgeTitle: i18n.tr("Favorite") | 253 | bottomEdgeTitle: i18n.tr("Favorite") |
80 | 250 | 254 | ||
81 | 251 | bottomEdgePageComponent: FavouritePage { | 255 | bottomEdgePageComponent: FavouritePage { |
82 | @@ -260,23 +264,27 @@ | |||
83 | 260 | states: [ | 264 | states: [ |
84 | 261 | State { | 265 | State { |
85 | 262 | name: "default" | 266 | name: "default" |
86 | 263 | StateChangeScript { | ||
87 | 264 | script: header.hide() | ||
88 | 265 | } | ||
89 | 266 | PropertyChanges { | 267 | PropertyChanges { |
90 | 267 | target: scrollableView | 268 | target: scrollableView |
91 | 268 | clip: false | 269 | clip: false |
92 | 269 | } | 270 | } |
93 | 271 | PropertyChanges { | ||
94 | 272 | target: calculatorPage.head | ||
95 | 273 | visible: false | ||
96 | 274 | preset: "" | ||
97 | 275 | } | ||
98 | 270 | }, | 276 | }, |
99 | 271 | State { | 277 | State { |
100 | 272 | name: "selection" | 278 | name: "selection" |
101 | 273 | StateChangeScript { | ||
102 | 274 | script: header.show() | ||
103 | 275 | } | ||
104 | 276 | PropertyChanges { | 279 | PropertyChanges { |
105 | 277 | target: scrollableView | 280 | target: scrollableView |
106 | 278 | clip: true | 281 | clip: true |
107 | 279 | } | 282 | } |
108 | 283 | PropertyChanges { | ||
109 | 284 | target: calculatorPage.head | ||
110 | 285 | visible: true | ||
111 | 286 | preset: "select" | ||
112 | 287 | } | ||
113 | 280 | } | 288 | } |
114 | 281 | ] | 289 | ] |
115 | 282 | 290 | ||
116 | @@ -289,52 +297,45 @@ | |||
117 | 289 | Keys.onPressed: textInputField.keyPress(event) | 297 | Keys.onPressed: textInputField.keyPress(event) |
118 | 290 | Keys.onReleased: textInputField.keyRelease(event) | 298 | Keys.onReleased: textInputField.keyRelease(event) |
119 | 291 | 299 | ||
164 | 292 | Header { | 300 | head.visible: false |
165 | 293 | id: header | 301 | head.locked: true |
166 | 294 | visible: true | 302 | head.backAction: Action { |
167 | 295 | useDeprecatedToolbar: false | 303 | objectName: "cancelSelectionAction" |
168 | 296 | property color dividerColor: "#babbbc" | 304 | iconName: "close" |
169 | 297 | property color panelColor: "white" | 305 | text: i18n.tr("Cancel") |
170 | 298 | config: PageHeadConfiguration { | 306 | onTriggered: visualModel.cancelSelection() |
171 | 299 | backAction: Action { | 307 | } |
172 | 300 | objectName: "cancelSelectionAction" | 308 | head.actions: [ |
173 | 301 | iconName: "close" | 309 | Action { |
174 | 302 | text: i18n.tr("Cancel") | 310 | id: selectAllAction |
175 | 303 | onTriggered: visualModel.cancelSelection() | 311 | objectName: "selectAllAction" |
176 | 304 | } | 312 | iconName: "select" |
177 | 305 | actions: [ | 313 | // Until a select none icon will be added to the theme we have to use |
178 | 306 | Action { | 314 | // our own |
179 | 307 | id: selectAllAction | 315 | iconSource: visualModel.selectedItems.count < visualModel.items.count ? |
180 | 308 | objectName: "selectAllAction" | 316 | Qt.resolvedUrl("graphics/select.svg") : |
181 | 309 | iconName: "select" | 317 | Qt.resolvedUrl("graphics/select_none.svg") |
182 | 310 | // Until a select none icon will be added to the theme we have to use | 318 | text: visualModel.selectedItems.count < visualModel.items.count ? |
183 | 311 | // our own | 319 | i18n.tr("Select All") : i18n.tr("Select None") |
184 | 312 | iconSource: visualModel.selectedItems.count < visualModel.items.count ? | 320 | onTriggered: visualModel.selectAll() |
185 | 313 | Qt.resolvedUrl("graphics/select.svg") : | 321 | }, |
186 | 314 | Qt.resolvedUrl("graphics/select_none.svg") | 322 | Action { |
187 | 315 | text: visualModel.selectedItems.count < visualModel.items.count ? | 323 | id: copySelectedAction |
188 | 316 | i18n.tr("Select All") : i18n.tr("Select None") | 324 | objectName: "copySelectedAction" |
189 | 317 | onTriggered: visualModel.selectAll() | 325 | iconName: "edit-copy" |
190 | 318 | }, | 326 | text: i18n.tr("Copy") |
191 | 319 | Action { | 327 | onTriggered: calculatorPage.copySelectedCalculations() |
192 | 320 | id: copySelectedAction | 328 | enabled: visualModel.selectedItems.count > 0 |
193 | 321 | objectName: "copySelectedAction" | 329 | }, |
194 | 322 | iconName: "edit-copy" | 330 | Action { |
195 | 323 | text: i18n.tr("Copy") | 331 | id: multiDeleteAction |
196 | 324 | onTriggered: calculatorPage.copySelectedCalculations() | 332 | objectName: "multiDeleteAction" |
197 | 325 | enabled: visualModel.selectedItems.count > 0 | 333 | iconName: "delete" |
198 | 326 | }, | 334 | text: i18n.tr("Delete") |
199 | 327 | Action { | 335 | onTriggered: calculatorPage.deleteSelectedCalculations() |
200 | 328 | id: multiDeleteAction | 336 | enabled: visualModel.selectedItems.count > 0 |
157 | 329 | objectName: "multiDeleteAction" | ||
158 | 330 | iconName: "delete" | ||
159 | 331 | text: i18n.tr("Delete") | ||
160 | 332 | onTriggered: calculatorPage.deleteSelectedCalculations() | ||
161 | 333 | enabled: visualModel.selectedItems.count > 0 | ||
162 | 334 | } | ||
163 | 335 | ] | ||
201 | 336 | } | 337 | } |
203 | 337 | } | 338 | ] |
204 | 338 | 339 | ||
205 | 339 | Component { | 340 | Component { |
206 | 340 | id: emptyDelegate | 341 | id: emptyDelegate |
207 | @@ -418,7 +419,7 @@ | |||
208 | 418 | Loader { | 419 | Loader { |
209 | 419 | id: screenDelegateFavouriteAction | 420 | id: screenDelegateFavouriteAction |
210 | 420 | sourceComponent: Action { | 421 | sourceComponent: Action { |
212 | 421 | iconName: (editedCalculationIndex == model.index || model.isFavourite) ? "starred" : "non-starred" | 422 | iconName: (mainView.editedCalculationIndex == model.index || model.isFavourite) ? "starred" : "non-starred" |
213 | 422 | 423 | ||
214 | 423 | text: i18n.tr("Add to favorites") | 424 | text: i18n.tr("Add to favorites") |
215 | 424 | onTriggered: { | 425 | onTriggered: { |
216 | @@ -530,11 +531,8 @@ | |||
217 | 530 | 531 | ||
218 | 531 | ScrollableView { | 532 | ScrollableView { |
219 | 532 | anchors { | 533 | anchors { |
222 | 533 | top: header.bottom | 534 | fill: parent |
221 | 534 | bottom: parent.bottom | ||
223 | 535 | bottomMargin: textInputField.visible ? 0 : -keyboardLoader.height | 535 | bottomMargin: textInputField.visible ? 0 : -keyboardLoader.height |
224 | 536 | left: parent.left | ||
225 | 537 | right: parent.right | ||
226 | 538 | } | 536 | } |
227 | 539 | id: scrollableView | 537 | id: scrollableView |
228 | 540 | objectName: "scrollableView" | 538 | objectName: "scrollableView" |
229 | @@ -738,4 +736,3 @@ | |||
230 | 738 | } | 736 | } |
231 | 739 | } | 737 | } |
232 | 740 | } | 738 | } |
233 | 741 | |||
234 | 742 | 739 | ||
235 | === modified file 'app/ui/CalcKeyboard.qml' | |||
236 | --- app/ui/CalcKeyboard.qml 2015-11-12 15:03:12 +0000 | |||
237 | +++ app/ui/CalcKeyboard.qml 2015-11-24 13:43:45 +0000 | |||
238 | @@ -1,5 +1,5 @@ | |||
239 | 1 | /* | 1 | /* |
241 | 2 | * Copyright (C) 2014 Canonical Ltd | 2 | * Copyright (C) 2014-2015 Canonical Ltd |
242 | 3 | * | 3 | * |
243 | 4 | * This file is part of Ubuntu Calculator App | 4 | * This file is part of Ubuntu Calculator App |
244 | 5 | * | 5 | * |
245 | @@ -15,8 +15,8 @@ | |||
246 | 15 | * You should have received a copy of the GNU General Public License | 15 | * You should have received a copy of the GNU General Public License |
247 | 16 | * along with this program. If not, see <http://www.gnu.org/licenses/>. | 16 | * along with this program. If not, see <http://www.gnu.org/licenses/>. |
248 | 17 | */ | 17 | */ |
251 | 18 | import QtQuick 2.3 | 18 | import QtQuick 2.4 |
252 | 19 | import Ubuntu.Components 1.1 | 19 | import Ubuntu.Components 1.3 |
253 | 20 | 20 | ||
254 | 21 | Rectangle { | 21 | Rectangle { |
255 | 22 | id: virtualKeyboard | 22 | id: virtualKeyboard |
256 | 23 | 23 | ||
257 | === modified file 'app/ui/FavouritePage.qml' | |||
258 | --- app/ui/FavouritePage.qml 2015-06-12 01:55:49 +0000 | |||
259 | +++ app/ui/FavouritePage.qml 2015-11-24 13:43:45 +0000 | |||
260 | @@ -15,10 +15,10 @@ | |||
261 | 15 | * You should have received a copy of the GNU General Public License | 15 | * You should have received a copy of the GNU General Public License |
262 | 16 | * along with this program. If not, see <http://www.gnu.org/licenses/>. | 16 | * along with this program. If not, see <http://www.gnu.org/licenses/>. |
263 | 17 | */ | 17 | */ |
265 | 18 | import QtQuick 2.3 | 18 | import QtQuick 2.4 |
266 | 19 | import QtQuick.Layouts 1.1 | 19 | import QtQuick.Layouts 1.1 |
269 | 20 | import Ubuntu.Components 1.1 | 20 | import Ubuntu.Components 1.3 |
270 | 21 | import Ubuntu.Components.ListItems 1.0 as ListItem | 21 | import Ubuntu.Components.ListItems 1.3 as ListItem |
271 | 22 | 22 | ||
272 | 23 | import "../engine" | 23 | import "../engine" |
273 | 24 | import "../upstreamcomponents" | 24 | import "../upstreamcomponents" |
274 | 25 | 25 | ||
275 | === modified file 'app/ui/KeyboardButton.qml' | |||
276 | --- app/ui/KeyboardButton.qml 2015-05-27 23:55:16 +0000 | |||
277 | +++ app/ui/KeyboardButton.qml 2015-11-24 13:43:45 +0000 | |||
278 | @@ -1,5 +1,5 @@ | |||
279 | 1 | /* | 1 | /* |
281 | 2 | * Copyright 2013 Canonical Ltd. | 2 | * Copyright 2013, 2015 Canonical Ltd. |
282 | 3 | * | 3 | * |
283 | 4 | * This file is part of ubuntu-calculator-app. | 4 | * This file is part of ubuntu-calculator-app. |
284 | 5 | * | 5 | * |
285 | @@ -16,8 +16,8 @@ | |||
286 | 16 | * along with this program. If not, see <http://www.gnu.org/licenses/>. | 16 | * along with this program. If not, see <http://www.gnu.org/licenses/>. |
287 | 17 | */ | 17 | */ |
288 | 18 | 18 | ||
291 | 19 | import QtQuick 2.3 | 19 | import QtQuick 2.4 |
292 | 20 | import Ubuntu.Components 1.1 | 20 | import Ubuntu.Components 1.3 |
293 | 21 | 21 | ||
294 | 22 | AbstractButton { | 22 | AbstractButton { |
295 | 23 | id: buttonRect | 23 | id: buttonRect |
296 | 24 | 24 | ||
297 | === modified file 'app/ui/KeyboardPage.qml' | |||
298 | --- app/ui/KeyboardPage.qml 2015-03-06 21:13:52 +0000 | |||
299 | +++ app/ui/KeyboardPage.qml 2015-11-24 13:43:45 +0000 | |||
300 | @@ -1,5 +1,22 @@ | |||
303 | 1 | import QtQuick 2.3 | 1 | /* |
304 | 2 | import Ubuntu.Components 1.1 | 2 | * Copyright (C) 2015 Canonical Ltd |
305 | 3 | * | ||
306 | 4 | * This file is part of Ubuntu Calculator App | ||
307 | 5 | * | ||
308 | 6 | * Ubuntu Calculator App is free software: you can redistribute it and/or modify | ||
309 | 7 | * it under the terms of the GNU General Public License version 3 as | ||
310 | 8 | * published by the Free Software Foundation. | ||
311 | 9 | * | ||
312 | 10 | * Ubuntu Calculator App is distributed in the hope that it will be useful, | ||
313 | 11 | * but WITHOUT ANY WARRANTY; without even the implied warranty of | ||
314 | 12 | * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the | ||
315 | 13 | * GNU General Public License for more details. | ||
316 | 14 | * | ||
317 | 15 | * You should have received a copy of the GNU General Public License | ||
318 | 16 | * along with this program. If not, see <http://www.gnu.org/licenses/>. | ||
319 | 17 | */ | ||
320 | 18 | import QtQuick 2.4 | ||
321 | 19 | import Ubuntu.Components 1.3 | ||
322 | 3 | 20 | ||
323 | 4 | Grid { | 21 | Grid { |
324 | 5 | id: keyboardRoot | 22 | id: keyboardRoot |
325 | 6 | 23 | ||
326 | === modified file 'app/ui/LandscapeKeyboard.qml' | |||
327 | --- app/ui/LandscapeKeyboard.qml 2015-06-11 15:40:14 +0000 | |||
328 | +++ app/ui/LandscapeKeyboard.qml 2015-11-24 13:43:45 +0000 | |||
329 | @@ -1,5 +1,23 @@ | |||
332 | 1 | import QtQuick 2.3 | 1 | /* |
333 | 2 | import Ubuntu.Components 1.1 | 2 | * Copyright (C) 2015 Canonical Ltd |
334 | 3 | * | ||
335 | 4 | * This file is part of Ubuntu Calculator App | ||
336 | 5 | * | ||
337 | 6 | * Ubuntu Calculator App is free software: you can redistribute it and/or modify | ||
338 | 7 | * it under the terms of the GNU General Public License version 3 as | ||
339 | 8 | * published by the Free Software Foundation. | ||
340 | 9 | * | ||
341 | 10 | * Ubuntu Calculator App is distributed in the hope that it will be useful, | ||
342 | 11 | * but WITHOUT ANY WARRANTY; without even the implied warranty of | ||
343 | 12 | * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the | ||
344 | 13 | * GNU General Public License for more details. | ||
345 | 14 | * | ||
346 | 15 | * You should have received a copy of the GNU General Public License | ||
347 | 16 | * along with this program. If not, see <http://www.gnu.org/licenses/>. | ||
348 | 17 | */ | ||
349 | 18 | |||
350 | 19 | import QtQuick 2.4 | ||
351 | 20 | import Ubuntu.Components 1.3 | ||
352 | 3 | 21 | ||
353 | 4 | CalcKeyboard { | 22 | CalcKeyboard { |
354 | 5 | id: calcKeyboard | 23 | id: calcKeyboard |
355 | 6 | 24 | ||
356 | === modified file 'app/ui/PortraitKeyboard.qml' | |||
357 | --- app/ui/PortraitKeyboard.qml 2015-06-16 10:26:39 +0000 | |||
358 | +++ app/ui/PortraitKeyboard.qml 2015-11-24 13:43:45 +0000 | |||
359 | @@ -1,5 +1,22 @@ | |||
362 | 1 | import QtQuick 2.3 | 1 | /* |
363 | 2 | import Ubuntu.Components 1.1 | 2 | * Copyright (C) 2015 Canonical Ltd |
364 | 3 | * | ||
365 | 4 | * This file is part of Ubuntu Calculator App | ||
366 | 5 | * | ||
367 | 6 | * Ubuntu Calculator App is free software: you can redistribute it and/or modify | ||
368 | 7 | * it under the terms of the GNU General Public License version 3 as | ||
369 | 8 | * published by the Free Software Foundation. | ||
370 | 9 | * | ||
371 | 10 | * Ubuntu Calculator App is distributed in the hope that it will be useful, | ||
372 | 11 | * but WITHOUT ANY WARRANTY; without even the implied warranty of | ||
373 | 12 | * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the | ||
374 | 13 | * GNU General Public License for more details. | ||
375 | 14 | * | ||
376 | 15 | * You should have received a copy of the GNU General Public License | ||
377 | 16 | * along with this program. If not, see <http://www.gnu.org/licenses/>. | ||
378 | 17 | */ | ||
379 | 18 | import QtQuick 2.4 | ||
380 | 19 | import Ubuntu.Components 1.3 | ||
381 | 3 | 20 | ||
382 | 4 | CalcKeyboard { | 21 | CalcKeyboard { |
383 | 5 | id: calcKeyboard | 22 | id: calcKeyboard |
384 | 6 | 23 | ||
385 | === modified file 'app/ui/Screen.qml' | |||
386 | --- app/ui/Screen.qml 2015-11-14 23:29:53 +0000 | |||
387 | +++ app/ui/Screen.qml 2015-11-24 13:43:45 +0000 | |||
388 | @@ -15,8 +15,8 @@ | |||
389 | 15 | * You should have received a copy of the GNU General Public License | 15 | * You should have received a copy of the GNU General Public License |
390 | 16 | * along with this program. If not, see <http://www.gnu.org/licenses/>. | 16 | * along with this program. If not, see <http://www.gnu.org/licenses/>. |
391 | 17 | */ | 17 | */ |
394 | 18 | import QtQuick 2.3 | 18 | import QtQuick 2.4 |
395 | 19 | import Ubuntu.Components 1.1 | 19 | import Ubuntu.Components 1.3 |
396 | 20 | 20 | ||
397 | 21 | import "../upstreamcomponents" | 21 | import "../upstreamcomponents" |
398 | 22 | import "../engine/formula.js" as Formula | 22 | import "../engine/formula.js" as Formula |
399 | 23 | 23 | ||
400 | === modified file 'app/ui/ScrollableView.qml' | |||
401 | --- app/ui/ScrollableView.qml 2015-02-13 22:20:01 +0000 | |||
402 | +++ app/ui/ScrollableView.qml 2015-11-24 13:43:45 +0000 | |||
403 | @@ -1,5 +1,5 @@ | |||
404 | 1 | /* | 1 | /* |
406 | 2 | * Copyright (C) 2014 Canonical Ltd | 2 | * Copyright (C) 2014-2015 Canonical Ltd |
407 | 3 | * | 3 | * |
408 | 4 | * This file is part of Ubuntu Calculator App | 4 | * This file is part of Ubuntu Calculator App |
409 | 5 | * | 5 | * |
410 | @@ -15,7 +15,7 @@ | |||
411 | 15 | * You should have received a copy of the GNU General Public License | 15 | * You should have received a copy of the GNU General Public License |
412 | 16 | * along with this program. If not, see <http://www.gnu.org/licenses/>. | 16 | * along with this program. If not, see <http://www.gnu.org/licenses/>. |
413 | 17 | */ | 17 | */ |
415 | 18 | import QtQuick 2.3 | 18 | import QtQuick 2.4 |
416 | 19 | 19 | ||
417 | 20 | Flickable { | 20 | Flickable { |
418 | 21 | id: flickableHistory | 21 | id: flickableHistory |
419 | 22 | 22 | ||
420 | === modified file 'app/upstreamcomponents/EmptyState.qml' | |||
421 | --- app/upstreamcomponents/EmptyState.qml 2015-03-06 18:09:49 +0000 | |||
422 | +++ app/upstreamcomponents/EmptyState.qml 2015-11-24 13:43:45 +0000 | |||
423 | @@ -1,5 +1,5 @@ | |||
426 | 1 | import QtQuick 2.0 | 1 | import QtQuick 2.4 |
427 | 2 | import Ubuntu.Components 1.1 | 2 | import Ubuntu.Components 1.3 |
428 | 3 | 3 | ||
429 | 4 | /* | 4 | /* |
430 | 5 | Component which displays an empty state (approved by design). It offers an | 5 | Component which displays an empty state (approved by design). It offers an |
431 | 6 | 6 | ||
432 | === modified file 'app/upstreamcomponents/ListItemWithActions.qml' | |||
433 | --- app/upstreamcomponents/ListItemWithActions.qml 2015-02-02 20:35:38 +0000 | |||
434 | +++ app/upstreamcomponents/ListItemWithActions.qml 2015-11-24 13:43:45 +0000 | |||
435 | @@ -1,5 +1,5 @@ | |||
436 | 1 | /* | 1 | /* |
438 | 2 | * Copyright (C) 2012-2014 Canonical, Ltd. | 2 | * Copyright (C) 2012-2015 Canonical, Ltd. |
439 | 3 | * | 3 | * |
440 | 4 | * This program is free software; you can redistribute it and/or modify | 4 | * This program is free software; you can redistribute it and/or modify |
441 | 5 | * it under the terms of the GNU General Public License as published by | 5 | * it under the terms of the GNU General Public License as published by |
442 | @@ -14,8 +14,8 @@ | |||
443 | 14 | * along with this program. If not, see <http://www.gnu.org/licenses/>. | 14 | * along with this program. If not, see <http://www.gnu.org/licenses/>. |
444 | 15 | */ | 15 | */ |
445 | 16 | 16 | ||
448 | 17 | import QtQuick 2.2 | 17 | import QtQuick 2.4 |
449 | 18 | import Ubuntu.Components 1.1 | 18 | import Ubuntu.Components 1.3 |
450 | 19 | 19 | ||
451 | 20 | Item { | 20 | Item { |
452 | 21 | id: root | 21 | id: root |
453 | 22 | 22 | ||
454 | === modified file 'app/upstreamcomponents/ListItemWithActionsCheckBox.qml' | |||
455 | --- app/upstreamcomponents/ListItemWithActionsCheckBox.qml 2014-12-03 22:54:49 +0000 | |||
456 | +++ app/upstreamcomponents/ListItemWithActionsCheckBox.qml 2015-11-24 13:43:45 +0000 | |||
457 | @@ -1,5 +1,5 @@ | |||
458 | 1 | /* | 1 | /* |
460 | 2 | * Copyright (C) 2012-2014 Canonical, Ltd. | 2 | * Copyright (C) 2012-2015 Canonical, Ltd. |
461 | 3 | * | 3 | * |
462 | 4 | * This program is free software; you can redistribute it and/or modify | 4 | * This program is free software; you can redistribute it and/or modify |
463 | 5 | * it under the terms of the GNU General Public License as published by | 5 | * it under the terms of the GNU General Public License as published by |
464 | @@ -14,8 +14,8 @@ | |||
465 | 14 | * along with this program. If not, see <http://www.gnu.org/licenses/>. | 14 | * along with this program. If not, see <http://www.gnu.org/licenses/>. |
466 | 15 | */ | 15 | */ |
467 | 16 | 16 | ||
470 | 17 | import QtQuick 2.2 | 17 | import QtQuick 2.4 |
471 | 18 | import Ubuntu.Components 1.1 | 18 | import Ubuntu.Components 1.3 |
472 | 19 | 19 | ||
473 | 20 | CheckBox { | 20 | CheckBox { |
474 | 21 | checked: root.selected | 21 | checked: root.selected |
475 | 22 | 22 | ||
476 | === modified file 'app/upstreamcomponents/MultipleSelectionVisualModel.qml' | |||
477 | --- app/upstreamcomponents/MultipleSelectionVisualModel.qml 2015-01-09 17:32:44 +0000 | |||
478 | +++ app/upstreamcomponents/MultipleSelectionVisualModel.qml 2015-11-24 13:43:45 +0000 | |||
479 | @@ -14,7 +14,7 @@ | |||
480 | 14 | * along with this program. If not, see <http://www.gnu.org/licenses/>. | 14 | * along with this program. If not, see <http://www.gnu.org/licenses/>. |
481 | 15 | */ | 15 | */ |
482 | 16 | 16 | ||
484 | 17 | import QtQuick 2.2 | 17 | import QtQuick 2.4 |
485 | 18 | 18 | ||
486 | 19 | VisualDataModel { | 19 | VisualDataModel { |
487 | 20 | id: visualModel | 20 | id: visualModel |
488 | 21 | 21 | ||
489 | === modified file 'app/upstreamcomponents/PageWithBottomEdge.qml' | |||
490 | --- app/upstreamcomponents/PageWithBottomEdge.qml 2015-01-28 21:32:05 +0000 | |||
491 | +++ app/upstreamcomponents/PageWithBottomEdge.qml 2015-11-24 13:43:45 +0000 | |||
492 | @@ -1,5 +1,5 @@ | |||
493 | 1 | /* | 1 | /* |
495 | 2 | * Copyright (C) 2014 Canonical, Ltd. | 2 | * Copyright (C) 2014-2015 Canonical, Ltd. |
496 | 3 | * | 3 | * |
497 | 4 | * This program is free software; you can redistribute it and/or modify | 4 | * This program is free software; you can redistribute it and/or modify |
498 | 5 | * it under the terms of the GNU General Public License as published by | 5 | * it under the terms of the GNU General Public License as published by |
499 | @@ -62,8 +62,8 @@ | |||
500 | 62 | 62 | ||
501 | 63 | */ | 63 | */ |
502 | 64 | 64 | ||
505 | 65 | import QtQuick 2.2 | 65 | import QtQuick 2.4 |
506 | 66 | import Ubuntu.Components 1.1 | 66 | import Ubuntu.Components 1.3 |
507 | 67 | 67 | ||
508 | 68 | Page { | 68 | Page { |
509 | 69 | id: page | 69 | id: page |
510 | @@ -105,7 +105,6 @@ | |||
511 | 105 | edgeLoader.item.active = true | 105 | edgeLoader.item.active = true |
512 | 106 | page.pageStack.push(edgeLoader.item) | 106 | page.pageStack.push(edgeLoader.item) |
513 | 107 | if (edgeLoader.item.flickable) { | 107 | if (edgeLoader.item.flickable) { |
514 | 108 | edgeLoader.item.flickable.contentY = -page.header.height | ||
515 | 109 | edgeLoader.item.flickable.returnToBounds() | 108 | edgeLoader.item.flickable.returnToBounds() |
516 | 110 | } | 109 | } |
517 | 111 | if (edgeLoader.item.ready) | 110 | if (edgeLoader.item.ready) |
518 | @@ -170,7 +169,7 @@ | |||
519 | 170 | z: 1 | 169 | z: 1 |
520 | 171 | width: tipLabel.paintedWidth + units.gu(6) | 170 | width: tipLabel.paintedWidth + units.gu(6) |
521 | 172 | height: bottomEdge.tipHeight + units.gu(1) | 171 | height: bottomEdge.tipHeight + units.gu(1) |
523 | 173 | color: Theme.palette.normal.overlay | 172 | backgroundColor: Theme.palette.normal.overlay |
524 | 174 | Label { | 173 | Label { |
525 | 175 | id: tipLabel | 174 | id: tipLabel |
526 | 176 | 175 | ||
527 | 177 | 176 | ||
528 | === modified file 'manifest.json.in' | |||
529 | --- manifest.json.in 2015-07-31 11:54:58 +0000 | |||
530 | +++ manifest.json.in 2015-11-24 13:43:45 +0000 | |||
531 | @@ -2,7 +2,7 @@ | |||
532 | 2 | "name": "com.ubuntu.calculator", | 2 | "name": "com.ubuntu.calculator", |
533 | 3 | "architecture": "all", | 3 | "architecture": "all", |
534 | 4 | "description": "Powerful and easy to use calculator.", | 4 | "description": "Powerful and easy to use calculator.", |
536 | 5 | "framework": "ubuntu-sdk-14.10", | 5 | "framework": "ubuntu-sdk-15.04.1-qml", |
537 | 6 | "hooks": { | 6 | "hooks": { |
538 | 7 | "calculator": { | 7 | "calculator": { |
539 | 8 | "apparmor": "@APP_HARDCODE@.apparmor", | 8 | "apparmor": "@APP_HARDCODE@.apparmor", |
PASSED: Continuous integration, rev:235 /core-apps- jenkins. ubuntu. com/job/ calculator- app-ci/ 11/ /core-apps- jenkins. ubuntu. com/job/ generic- update- mp/118/ console
https:/
Executed test runs:
None: https:/
Click here to trigger a rebuild: /core-apps- jenkins. ubuntu. com/job/ calculator- app-ci/ 11/rebuild
https:/