Merge lp:~tpeeters/ubuntu-ui-toolkit/fixDoubleLine into lp:ubuntu-ui-toolkit/staging

Proposed by Tim Peeters
Status: Merged
Approved by: Zsombor Egri
Approved revision: 1799
Merged at revision: 1799
Proposed branch: lp:~tpeeters/ubuntu-ui-toolkit/fixDoubleLine
Merge into: lp:ubuntu-ui-toolkit/staging
Diff against target: 33 lines (+8/-1)
2 files modified
src/Ubuntu/Components/1.3/MainView.qml (+2/-1)
tests/unit_x11/tst_components/tst_pagestack.qml (+6/-0)
To merge this branch: bzr merge lp:~tpeeters/ubuntu-ui-toolkit/fixDoubleLine
Reviewer Review Type Date Requested Status
Zsombor Egri Approve
Renato Araujo Oliveira Filho (community) Approve
PS Jenkins bot continuous-integration Approve
Review via email: mp+281876@code.launchpad.net

Commit message

Hide AppHeader when using AdaptivePageLayout. Fixes bug 1531871.

To post a comment you must log in.
1799. By Tim Peeters

fix pagestack unit test

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
Renato Araujo Oliveira Filho (renatofilho) wrote :

I can confirm that fix the address book problem.

review: Approve
Revision history for this message
Zsombor Egri (zsombi) wrote :

Nice thanks!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/Ubuntu/Components/1.3/MainView.qml'
2--- src/Ubuntu/Components/1.3/MainView.qml 2016-01-06 11:24:54 +0000
3+++ src/Ubuntu/Components/1.3/MainView.qml 2016-01-07 14:55:46 +0000
4@@ -162,7 +162,8 @@
5 // don't show the application header if the page has its own header.
6 visible: !(internal.activePage &&
7 internal.activePage.hasOwnProperty("header") &&
8- internal.activePage.header)
9+ internal.activePage.header) &&
10+ internal.activePage
11
12 height: visible ? implicitHeight : 0
13
14
15=== modified file 'tests/unit_x11/tst_components/tst_pagestack.qml'
16--- tests/unit_x11/tst_components/tst_pagestack.qml 2015-10-21 19:33:28 +0000
17+++ tests/unit_x11/tst_components/tst_pagestack.qml 2016-01-07 14:55:46 +0000
18@@ -54,9 +54,15 @@
19 id: tabs
20 Tab {
21 id: tab1
22+ page: Page {
23+ title: "Tab 1"
24+ }
25 }
26 Tab {
27 id: tab2
28+ page: Page {
29+ title: "Tab 2"
30+ }
31 }
32 }
33 Page {

Subscribers

People subscribed via source and target branches