Merge lp:~nik90/ubuntu-clock-app/fix-warning-message into lp:ubuntu-clock-app

Proposed by Nekhelesh Ramananthan
Status: Merged
Approved by: Nekhelesh Ramananthan
Approved revision: 142
Merged at revision: 141
Proposed branch: lp:~nik90/ubuntu-clock-app/fix-warning-message
Merge into: lp:ubuntu-clock-app
Diff against target: 31 lines (+2/-1)
2 files modified
app/alarm/AlarmPage.qml (+1/-1)
debian/changelog (+1/-0)
To merge this branch: bzr merge lp:~nik90/ubuntu-clock-app/fix-warning-message
Reviewer Review Type Date Requested Status
Ubuntu Phone Apps Jenkins Bot continuous-integration Approve
Kill Animals (community) Approve
Ubuntu Clock Developers Pending
Review via email: mp+238049@code.launchpad.net

Commit message

Fixed the warning related to undefined width in the AlarmPage.qml and corrected the delete icon's right margin in the alarm multiselect mode.

Description of the change

This MP implements the following,
- Fixes the warning related to undefined width in the AlarmPage.qml
- Corrects the delete icon's right margin in the alarm multiselect mode

Incorrect right margin screenshot
-> https://imgur.com/Ypf9a3X

Correct right margin screenshot implemented in this MP
-> https://imgur.com/r7zcHhR

To post a comment you must log in.
Revision history for this message
Kill Animals (kill-animals) wrote :
Download full text (3.6 KiB)

margin looks fine

qml: DEPRECATED use of Panel.opened property. This property will be made read-only,
                please use the opened property of the Page tools or use Panel.open() and Panel.close().
qml: Debug: Queue: Now has: 0 tracks
qml: Shuffle: true
qml: Debug: onCountChanged: 0
APP_ID isn't set, the handler can not be registered
UCUriHandler: Empty "APP_ID" environment variable, ignoring.
qml: WARNING: Sheets are deprecated. Consider using Dialog, Popover or PageStack instead.
qml: Debug: Version 1.2
qml: Debug: Arguments on startup: Debug: false
qml: Debug: Arguments on startup: Debug: false and file:
qml: INITIALIZED in tracks
qml: Playlist DB is version 1.3
qml: DB: Restore {}
qml: called LibraryListModel::filterRecent()
qml: Time:2014-10-11T00:21:10.111, Key:Angel Clare, Title:Angel Clare, Title2:Art Garfunkel, Type:album, Art:image://albumart/artist=Art%20Garfunkel&album=Angel%20Clare
qml: Time:2014-10-10T22:02:26.047, Key:hoieh, Title:hoieh, Title2:Playlist, Type:playlist, Art:
qml: Time:2014-10-10T21:47:23.506, Key:, Title:, Title2:, Type:album, Art:image://albumart/artist=&album=
D-Bus error: "Could not get thumbnail"
qml: {"art":"image://albumart/artist=Art%20Garfunkel&album=Angel%20Clare","key":"Angel Clare","time":"2014-10-11T00:21:10.111","title":"Angel Clare","title2":"Art Garfunkel","type":"album"}
APP_ID isn't set, the handler ignored
qml: {"key":"hoieh","time":"2014-10-10T22:02:26.047","title":"hoieh","title2":"Playlist","type":"playlist"}
qml: {"art":"image://albumart/artist=&album=","key":"Unknown Album","time":"2014-10-10T21:47:23.506","title":"Unknown Album","title2":"Unknown Artist","type":"album"}
qml: Debug: Queue: Now has: 1 tracks
qml: Debug: MusicQueue update currentIndex:
qml: Source: file:///home/akiva/Music/Art Garfunkel - Travelling Boy.mp3
qml: Index: 0
qml: RECENT Angel ClareAngel ClareArt Garfunkelimage://albumart/artist=Art%20Garfunkel&album=Angel%20Clare
qml: RECENT add OK
qml: called LibraryListModel::filterRecent()
qml: Time:2014-10-11T00:30:29.888, Key:Angel Clare, Title:Angel Clare, Title2:Art Garfunkel, Type:album, Art:image://albumart/artist=Art%20Garfunkel&album=Angel%20Clare
qml: Time:2014-10-10T22:02:26.047, Key:hoieh, Title:hoieh, Title2:Playlist, Type:playlist, Art:
qml: Time:2014-10-10T21:47:23.506, Key:, Title:, Title2:, Type:album, Art:image://albumart/artist=&album=
qml: {"art":"image://albumart/artist=Art%20Garfunkel&album=Angel%20Clare","key":"Angel Clare","time":"2014-10-11T00:30:29.888","title":"Angel Clare","title2":"Art Garfunkel","type":"album"}
qml: Debug: Queue: Now has: 2 tracks
ShaderEffect: Property 'source' is not assigned a valid texture provider (QQuickItem*).
ShaderEffect: Property 'source' is not assigned a valid texture provider (QQuickItem*).
qml: {"key":"hoieh","time":"2014-10-10T22:02:26.047","title":"hoieh","title2":"Playlist","type":"playlist"}
qml: {"art":"image://albumart/artist=&album=","key":"Unknown Album","time":"2014-10-10T21:47:23.506","title":"Unknown Album","title2":"Unknown Artist","type":"album"}
APP_ID isn't set, the handler ignored
qml: Increment UserMetrics
APP_ID isn't set, the handler ignored
file:///home/akiva/Programming/Deleteme/remix...

Read more...

review: Approve
Revision history for this message
Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) wrote :
review: Approve (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'app/alarm/AlarmPage.qml'
2--- app/alarm/AlarmPage.qml 2014-10-05 15:33:27 +0000
3+++ app/alarm/AlarmPage.qml 2014-10-11 10:24:47 +0000
4@@ -66,7 +66,6 @@
5 contents: Item {
6 anchors.right: parent ? parent.right: undefined
7 height: parent ? parent.height : undefined
8- width: childrenRect.width
9
10 HeaderButton {
11 id: selectButton
12@@ -105,6 +104,7 @@
13 id: deleteButton
14
15 anchors.right: parent.right
16+ anchors.rightMargin: units.gu(2)
17
18 iconName: "delete"
19 text: i18n.tr("Delete")
20
21=== modified file 'debian/changelog'
22--- debian/changelog 2014-10-10 10:19:10 +0000
23+++ debian/changelog 2014-10-11 10:24:47 +0000
24@@ -10,6 +10,7 @@
25 only. (LP: #1377538)
26 * Tweaked bottom edge to show the time to the next active alarm (LP: #1290793)
27 * Synced ListItemWithActions with upstream
28+ * Fixed alarm page header width warning and corrected icon spacing.
29
30 [Akiva Shammai Avraham]
31 * Improved the analog clock performance by updating the clock hands every second

Subscribers

People subscribed via source and target branches