Merge lp:~nik90/podbird/fix-header-issue into lp:podbird

Proposed by Nekhelesh Ramananthan
Status: Merged
Merged at revision: 67
Proposed branch: lp:~nik90/podbird/fix-header-issue
Merge into: lp:podbird
Diff against target: 116 lines (+16/-15)
4 files modified
app/podbird.qml (+0/-6)
app/ui/EpisodesPage.qml (+4/-4)
app/ui/PodcastsTab.qml (+8/-1)
po/com.mikeasoft.podbird.pot (+4/-4)
To merge this branch: bzr merge lp:~nik90/podbird/fix-header-issue
Reviewer Review Type Date Requested Status
Michael Sheldon Approve
Review via email: mp+256805@code.launchpad.net

Description of the change

This MP fixes,
- first listitem behind hidden by the header
- scrollbar in episodes page sometimes showing behind the page header (in vivid)
- Removes loader anchors as specified in the SDK docs (no change visually)

This is only a temporary fix to the listitem issue since it looks like a sdk issue. I tried creating a sample app to reproduce it but in vain. I will keep trying.

To post a comment you must log in.
Revision history for this message
Michael Sheldon (michael-sheldon) wrote :

Merged, but I've restored the anchors to the loader for now as it broke the settings page and search page on the desktop.

review: Approve
Revision history for this message
Nekhelesh Ramananthan (nik90) wrote :

Oh..that's strange. I will make sure to test on the desktop here on for stuff like these.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'app/podbird.qml'
2--- app/podbird.qml 2015-04-19 13:13:25 +0000
3+++ app/podbird.qml 2015-04-20 12:18:35 +0000
4@@ -226,9 +226,6 @@
5
6 page: Loader {
7 parent: searchTab
8- anchors.left: parent.left
9- anchors.right: parent.right
10- anchors.bottom: parent.bottom
11 source: (tabs.selectedTab === searchTab) ? Qt.resolvedUrl("ui/SearchPage.qml") : ""
12 }
13 }
14@@ -240,9 +237,6 @@
15
16 page: Loader {
17 parent: settingsTab
18- anchors.left: parent.left
19- anchors.right: parent.right
20- anchors.bottom: parent.bottom
21 source: (tabs.selectedTab === settingsTab) ? Qt.resolvedUrl("ui/SettingsPage.qml") : ""
22 }
23 }
24
25=== modified file 'app/ui/EpisodesPage.qml'
26--- app/ui/EpisodesPage.qml 2015-04-19 13:10:27 +0000
27+++ app/ui/EpisodesPage.qml 2015-04-20 12:18:35 +0000
28@@ -598,11 +598,11 @@
29 refreshing: episodesUpdating
30 onRefresh: updateEpisodesDatabase();
31 }
32- }
33
34- // #FIXME: Use SDK Scrollbar when it is themeable
35- CustomScrollBar {
36- listview: episodeList
37+ // #FIXME: Use SDK Scrollbar when it is themeable
38+ CustomScrollBar {
39+ listview: episodeList
40+ }
41 }
42
43 function refreshModel() {
44
45=== modified file 'app/ui/PodcastsTab.qml'
46--- app/ui/PodcastsTab.qml 2015-04-18 22:44:02 +0000
47+++ app/ui/PodcastsTab.qml 2015-04-20 12:18:35 +0000
48@@ -36,7 +36,12 @@
49 page: Page {
50 id: podcastPage
51
52- flickable: viewLoader.item
53+ /*
54+ #FIXME: The page flickable is to null instead of viewLoader.item since
55+ it otherwise creates bug http://pad.lv/1446162 which can confuse a
56+ new user.
57+ */
58+ flickable: null
59
60 /*
61 #FIXME: The following lines of code is necessary due to a upstream bug
62@@ -145,6 +150,7 @@
63
64 CardView {
65 id: cardView
66+ clip: true
67 model: sortedPodcastModel
68 delegate: Card {
69 id: albumCard
70@@ -177,6 +183,7 @@
71 flickDeceleration = flickDeceleration * scaleFactor;
72 }
73
74+ clip: true
75 model: sortedPodcastModel
76 anchors.fill: parent
77
78
79=== modified file 'po/com.mikeasoft.podbird.pot'
80--- po/com.mikeasoft.podbird.pot 2015-04-19 23:37:08 +0000
81+++ po/com.mikeasoft.podbird.pot 2015-04-20 12:18:35 +0000
82@@ -8,7 +8,7 @@
83 msgstr ""
84 "Project-Id-Version: \n"
85 "Report-Msgid-Bugs-To: \n"
86-"POT-Creation-Date: 2015-04-20 00:36+0100\n"
87+"POT-Creation-Date: 2015-04-20 12:09+0000\n"
88 "PO-Revision-Date: YEAR-MO-DA HO:MI+ZONE\n"
89 "Last-Translator: FULL NAME <EMAIL@ADDRESS>\n"
90 "Language-Team: LANGUAGE <LL@li.org>\n"
91@@ -31,7 +31,7 @@
92 msgid "Add New Podcasts"
93 msgstr ""
94
95-#: ../app/podbird.qml:239
96+#: ../app/podbird.qml:236
97 msgid "Settings"
98 msgstr ""
99
100@@ -267,7 +267,7 @@
101 msgid "No podcasts found matching the search term."
102 msgstr ""
103
104-#: ../app/ui/PodcastsTab.qml:153 ../app/ui/PodcastsTab.qml:273
105+#: ../app/ui/PodcastsTab.qml:154 ../app/ui/PodcastsTab.qml:275
106 #, qt-format
107 msgid "%1 unheard episode"
108 msgid_plural "%1 unheard episodes"
109@@ -493,6 +493,6 @@
110 msgid "Finish"
111 msgstr ""
112
113-#: /home/mike/src/podbird/build-debian-packaging-Ubuntu_Device3_GCC_armhf_ubuntu_sdk_14_10_utopic-Default/po/Podbird.desktop.in.h:1
114+#: /home/krnekhelesh/Documents/Ubuntu-Projects/MP-Reviews/builddir/build-fix-header-issue-UbuntuSDK_for_armhf_GCC_ubuntu_sdk_15_04_vivid-Default/po/Podbird.desktop.in.h:1
115 msgid "Podbird"
116 msgstr ""

Subscribers

People subscribed via source and target branches