Merge lp:~ubuntu-clock-dev/ubuntu-clock-app/2-migrate-add-city-listitems into lp:ubuntu-clock-app

Proposed by Nekhelesh Ramananthan
Status: Merged
Approved by: Bartosz Kosiorek
Approved revision: 273
Merged at revision: 309
Proposed branch: lp:~ubuntu-clock-dev/ubuntu-clock-app/2-migrate-add-city-listitems
Merge into: lp:ubuntu-clock-app
Prerequisite: lp:~ubuntu-clock-dev/ubuntu-clock-app/1-migrate-to-15.04-framework
Diff against target: 39 lines (+15/-5)
1 file modified
app/worldclock/WorldCityList.qml (+15/-5)
To merge this branch: bzr merge lp:~ubuntu-clock-dev/ubuntu-clock-app/2-migrate-add-city-listitems
Reviewer Review Type Date Requested Status
Bartosz Kosiorek Approve
Ubuntu Phone Apps Jenkins Bot continuous-integration Approve
Review via email: mp+263887@code.launchpad.net

Commit message

Migrate World City page listitems to the new 15.04 listitems

Description of the change

Migrate World City page listitems to the new 15.04 listitems

#blocked

Blocked due to,

1. Pre-requisite branch needs to go in first
2. There is a bug where deleting a world city does not remove the listitem when the world city count is 1.
3. Autopilot tests and qml tests needs to be updated.

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: Needs Fixing (continuous-integration)
273. By Nekhelesh Ramananthan

Merged lp:ubuntu-clock-app and resolved conflicts

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
Bartosz Kosiorek (gang65) wrote :

lgtm

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'app/worldclock/WorldCityList.qml'
--- app/worldclock/WorldCityList.qml 2015-06-18 22:57:49 +0000
+++ app/worldclock/WorldCityList.qml 2015-07-24 13:17:12 +0000
@@ -19,7 +19,6 @@
19import QtQuick 2.419import QtQuick 2.4
20import Timezone 1.020import Timezone 1.0
21import Ubuntu.Components 1.221import Ubuntu.Components 1.2
22import Ubuntu.Components.ListItems 1.0 as ListItem
23import "../components"22import "../components"
24import "../upstreamcomponents"23import "../upstreamcomponents"
2524
@@ -304,12 +303,23 @@
304 section.criteria: ViewSection.FirstCharacter303 section.criteria: ViewSection.FirstCharacter
305 section.labelPositioning: ViewSection.InlineLabels304 section.labelPositioning: ViewSection.InlineLabels
306305
307 section.delegate: ListItem.Header {306 section.delegate: ListItem {
308 text: section307 height: header.implicitHeight + units.gu(2)
308 Label {
309 id: header
310 text: section
311 font.weight: Font.DemiBold
312 anchors {
313 left: parent.left
314 leftMargin: units.gu(2)
315 verticalCenter: parent.verticalCenter
316 }
317 }
309 }318 }
310319
311 delegate: ListItem.Empty {320 delegate: ListItem {
312 showDivider: false321 height: worldCityDelegateColumn.height + units.gu(2)
322 divider.visible: false
313 objectName: "defaultWorldCityItem" + index323 objectName: "defaultWorldCityItem" + index
314324
315 Column {325 Column {

Subscribers

People subscribed via source and target branches

to all changes: