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
=== modified file 'app/podbird.qml'
--- app/podbird.qml 2015-04-19 13:13:25 +0000
+++ app/podbird.qml 2015-04-20 12:18:35 +0000
@@ -226,9 +226,6 @@
226226
227 page: Loader {227 page: Loader {
228 parent: searchTab228 parent: searchTab
229 anchors.left: parent.left
230 anchors.right: parent.right
231 anchors.bottom: parent.bottom
232 source: (tabs.selectedTab === searchTab) ? Qt.resolvedUrl("ui/SearchPage.qml") : ""229 source: (tabs.selectedTab === searchTab) ? Qt.resolvedUrl("ui/SearchPage.qml") : ""
233 }230 }
234 }231 }
@@ -240,9 +237,6 @@
240237
241 page: Loader {238 page: Loader {
242 parent: settingsTab239 parent: settingsTab
243 anchors.left: parent.left
244 anchors.right: parent.right
245 anchors.bottom: parent.bottom
246 source: (tabs.selectedTab === settingsTab) ? Qt.resolvedUrl("ui/SettingsPage.qml") : ""240 source: (tabs.selectedTab === settingsTab) ? Qt.resolvedUrl("ui/SettingsPage.qml") : ""
247 }241 }
248 }242 }
249243
=== modified file 'app/ui/EpisodesPage.qml'
--- app/ui/EpisodesPage.qml 2015-04-19 13:10:27 +0000
+++ app/ui/EpisodesPage.qml 2015-04-20 12:18:35 +0000
@@ -598,11 +598,11 @@
598 refreshing: episodesUpdating598 refreshing: episodesUpdating
599 onRefresh: updateEpisodesDatabase();599 onRefresh: updateEpisodesDatabase();
600 }600 }
601 }
602601
603 // #FIXME: Use SDK Scrollbar when it is themeable602 // #FIXME: Use SDK Scrollbar when it is themeable
604 CustomScrollBar {603 CustomScrollBar {
605 listview: episodeList604 listview: episodeList
605 }
606 }606 }
607607
608 function refreshModel() {608 function refreshModel() {
609609
=== modified file 'app/ui/PodcastsTab.qml'
--- app/ui/PodcastsTab.qml 2015-04-18 22:44:02 +0000
+++ app/ui/PodcastsTab.qml 2015-04-20 12:18:35 +0000
@@ -36,7 +36,12 @@
36 page: Page {36 page: Page {
37 id: podcastPage37 id: podcastPage
3838
39 flickable: viewLoader.item39 /*
40 #FIXME: The page flickable is to null instead of viewLoader.item since
41 it otherwise creates bug http://pad.lv/1446162 which can confuse a
42 new user.
43 */
44 flickable: null
4045
41 /*46 /*
42 #FIXME: The following lines of code is necessary due to a upstream bug47 #FIXME: The following lines of code is necessary due to a upstream bug
@@ -145,6 +150,7 @@
145150
146 CardView {151 CardView {
147 id: cardView152 id: cardView
153 clip: true
148 model: sortedPodcastModel154 model: sortedPodcastModel
149 delegate: Card {155 delegate: Card {
150 id: albumCard156 id: albumCard
@@ -177,6 +183,7 @@
177 flickDeceleration = flickDeceleration * scaleFactor;183 flickDeceleration = flickDeceleration * scaleFactor;
178 }184 }
179185
186 clip: true
180 model: sortedPodcastModel187 model: sortedPodcastModel
181 anchors.fill: parent188 anchors.fill: parent
182189
183190
=== modified file 'po/com.mikeasoft.podbird.pot'
--- po/com.mikeasoft.podbird.pot 2015-04-19 23:37:08 +0000
+++ po/com.mikeasoft.podbird.pot 2015-04-20 12:18:35 +0000
@@ -8,7 +8,7 @@
8msgstr ""8msgstr ""
9"Project-Id-Version: \n"9"Project-Id-Version: \n"
10"Report-Msgid-Bugs-To: \n"10"Report-Msgid-Bugs-To: \n"
11"POT-Creation-Date: 2015-04-20 00:36+0100\n"11"POT-Creation-Date: 2015-04-20 12:09+0000\n"
12"PO-Revision-Date: YEAR-MO-DA HO:MI+ZONE\n"12"PO-Revision-Date: YEAR-MO-DA HO:MI+ZONE\n"
13"Last-Translator: FULL NAME <EMAIL@ADDRESS>\n"13"Last-Translator: FULL NAME <EMAIL@ADDRESS>\n"
14"Language-Team: LANGUAGE <LL@li.org>\n"14"Language-Team: LANGUAGE <LL@li.org>\n"
@@ -31,7 +31,7 @@
31msgid "Add New Podcasts"31msgid "Add New Podcasts"
32msgstr ""32msgstr ""
3333
34#: ../app/podbird.qml:23934#: ../app/podbird.qml:236
35msgid "Settings"35msgid "Settings"
36msgstr ""36msgstr ""
3737
@@ -267,7 +267,7 @@
267msgid "No podcasts found matching the search term."267msgid "No podcasts found matching the search term."
268msgstr ""268msgstr ""
269269
270#: ../app/ui/PodcastsTab.qml:153 ../app/ui/PodcastsTab.qml:273270#: ../app/ui/PodcastsTab.qml:154 ../app/ui/PodcastsTab.qml:275
271#, qt-format271#, qt-format
272msgid "%1 unheard episode"272msgid "%1 unheard episode"
273msgid_plural "%1 unheard episodes"273msgid_plural "%1 unheard episodes"
@@ -493,6 +493,6 @@
493msgid "Finish"493msgid "Finish"
494msgstr ""494msgstr ""
495495
496#: /home/mike/src/podbird/build-debian-packaging-Ubuntu_Device3_GCC_armhf_ubuntu_sdk_14_10_utopic-Default/po/Podbird.desktop.in.h:1496#: /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
497msgid "Podbird"497msgid "Podbird"
498msgstr ""498msgstr ""

Subscribers

People subscribed via source and target branches