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
1=== modified file 'app/worldclock/WorldCityList.qml'
2--- app/worldclock/WorldCityList.qml 2015-06-18 22:57:49 +0000
3+++ app/worldclock/WorldCityList.qml 2015-07-24 13:17:12 +0000
4@@ -19,7 +19,6 @@
5 import QtQuick 2.4
6 import Timezone 1.0
7 import Ubuntu.Components 1.2
8-import Ubuntu.Components.ListItems 1.0 as ListItem
9 import "../components"
10 import "../upstreamcomponents"
11
12@@ -304,12 +303,23 @@
13 section.criteria: ViewSection.FirstCharacter
14 section.labelPositioning: ViewSection.InlineLabels
15
16- section.delegate: ListItem.Header {
17- text: section
18+ section.delegate: ListItem {
19+ height: header.implicitHeight + units.gu(2)
20+ Label {
21+ id: header
22+ text: section
23+ font.weight: Font.DemiBold
24+ anchors {
25+ left: parent.left
26+ leftMargin: units.gu(2)
27+ verticalCenter: parent.verticalCenter
28+ }
29+ }
30 }
31
32- delegate: ListItem.Empty {
33- showDivider: false
34+ delegate: ListItem {
35+ height: worldCityDelegateColumn.height + units.gu(2)
36+ divider.visible: false
37 objectName: "defaultWorldCityItem" + index
38
39 Column {

Subscribers

People subscribed via source and target branches

to all changes: