Merge lp:~ken-vandine/friends-app/page_for_flickable into lp:friends-app

Proposed by Ken VanDine
Status: Merged
Approved by: Florian Boucault
Approved revision: 85
Merged at revision: 84
Proposed branch: lp:~ken-vandine/friends-app/page_for_flickable
Merge into: lp:friends-app
Diff against target: 69 lines (+16/-14)
2 files modified
qml/TimeLine.qml (+10/-4)
qml/friends-app.qml (+6/-10)
To merge this branch: bzr merge lp:~ken-vandine/friends-app/page_for_flickable
Reviewer Review Type Date Requested Status
PS Jenkins bot (community) continuous-integration Approve
Super Friends Pending
Review via email: mp+175114@code.launchpad.net

Commit message

Make TimeLine a Page and set flickable to the listview to fix issues with the header

Description of the change

Make TimeLine a Page and set flickable to the listview to fix issues with the header

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) 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 'qml/TimeLine.qml'
2--- qml/TimeLine.qml 2013-04-11 15:39:15 +0000
3+++ qml/TimeLine.qml 2013-07-16 19:55:29 +0000
4@@ -21,11 +21,13 @@
5 import Ubuntu.Components.ListItems 0.1
6 import Friends 0.1
7
8-Rectangle {
9+Page {
10+ title: i18n.tr("Timeline")
11+ flickable: listView
12+
13 property string stream: ""
14 property int unseen: 0
15 property string search
16- color: "#f8f8f8"
17
18 /* Jump to the bottom of the view and set the currentIndex */
19 function jumpToBottom () {
20@@ -117,8 +119,12 @@
21
22 preferredHighlightBegin: 0
23 preferredHighlightEnd: preferredHighlightBegin
24- highlightRangeMode: ListView.StrictlyEnforceRange
25- //highlight: Rectangle { color: "black"; opacity: 0.1 }
26+ /* We need highlightRangeMode for keyboard navigation,
27+ * but it forces the page header to stay hidden
28+ * https://bugs.launchpad.net/ubuntu-ui-toolkit/+bug/1201936
29+ *
30+ * highlightRangeMode: ListView.StrictlyEnforceRange
31+ */
32
33 onCurrentIndexChanged: {
34 if (currentIndex == 0) {
35
36=== modified file 'qml/friends-app.qml'
37--- qml/friends-app.qml 2013-06-26 20:06:39 +0000
38+++ qml/friends-app.qml 2013-07-16 19:55:29 +0000
39@@ -46,7 +46,7 @@
40 anchors.fill: parent
41 Component.onCompleted: {
42 console.log ("accounts.count: "+accounts.count);
43- pageStack.push(timelinePage);
44+ pageStack.push(timeline);
45 if (accounts.count < 1)
46 pageStack.push(noAccountsPage);
47 }
48@@ -79,16 +79,12 @@
49 }
50 }
51
52- Page {
53- id: timelinePage
54- title: i18n.tr("Timeline")
55+ TimeLine {
56+ id: timeline
57+ anchors {
58+ fill: parent
59+ }
60 visible: false
61- TimeLine {
62- id: timeline
63- anchors {
64- fill: parent
65- }
66- }
67 tools: ToolbarItems {
68 ToolbarButton {
69 text: i18n.tr("Top")

Subscribers

People subscribed via source and target branches

to all changes: