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

Proposed by Tim Peeters
Status: Merged
Approved by: Cris Dywan
Approved revision: 2000
Merged at revision: 1998
Proposed branch: lp:~tpeeters/ubuntu-ui-toolkit/deprecateSubHeader
Merge into: lp:ubuntu-ui-toolkit/staging
Diff against target: 48 lines (+14/-0)
3 files modified
src/Ubuntu/Components/1.3/AdaptivePageLayout.qml (+3/-0)
src/Ubuntu/Components/Styles/1.3/PageHeadStyle.qml (+1/-0)
src/Ubuntu/Components/Themes/Ambiance/1.3/PageHeadStyle.qml (+10/-0)
To merge this branch: bzr merge lp:~tpeeters/ubuntu-ui-toolkit/deprecateSubHeader
Reviewer Review Type Date Requested Status
ubuntu-sdk-build-bot continuous-integration Approve
Cris Dywan Approve
Review via email: mp+296919@code.launchpad.net

Commit message

Draw a red outline around the subheader in AdaptivePageLayout to warn that the old header is deprecated.

Description of the change

As announced on the ubuntu-phone mailing list. PageHeader should be used instead of the Page.head property.

You can use tst_multicolumnheader.qml to test it.

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

revert change to export_qml_dir.sh

2000. By Tim Peeters

capitalization in comment

Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
Cris Dywan (kalikiana) wrote :

I like it, not overly intrusive but clearly visible, and simple implementation.

review: Approve
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-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 'src/Ubuntu/Components/1.3/AdaptivePageLayout.qml'
2--- src/Ubuntu/Components/1.3/AdaptivePageLayout.qml 2016-05-25 12:48:10 +0000
3+++ src/Ubuntu/Components/1.3/AdaptivePageLayout.qml 2016-06-09 10:38:15 +0000
4@@ -827,6 +827,9 @@
5 property color panelColor: layout.__propagated.header.panelColor
6 property color backgroundColor: layout.__propagated.header.backgroundColor
7
8+ // Enable red outline in the PageHeadStyle. See bug #1583636.
9+ property bool showDeprecatedWarning: true
10+
11 visible: !customHeader && holder.pageWrapper && holder.pageWrapper.active
12
13 // The multiColumn, page and showBackButton properties are used in
14
15=== modified file 'src/Ubuntu/Components/Styles/1.3/PageHeadStyle.qml'
16--- src/Ubuntu/Components/Styles/1.3/PageHeadStyle.qml 2015-09-22 14:39:47 +0000
17+++ src/Ubuntu/Components/Styles/1.3/PageHeadStyle.qml 2016-06-09 10:38:15 +0000
18@@ -17,6 +17,7 @@
19
20 /*!
21 \qmltype PageHeadStyle
22+ \deprecated
23 \inqmlmodule Ubuntu.Components.Styles 1.1
24 \ingroup style-api
25 \brief Style API for page header.
26
27=== modified file 'src/Ubuntu/Components/Themes/Ambiance/1.3/PageHeadStyle.qml'
28--- src/Ubuntu/Components/Themes/Ambiance/1.3/PageHeadStyle.qml 2016-02-29 16:02:50 +0000
29+++ src/Ubuntu/Components/Themes/Ambiance/1.3/PageHeadStyle.qml 2016-06-09 10:38:15 +0000
30@@ -50,8 +50,18 @@
31 */
32 property color backgroundColor: styledItem.backgroundColor
33 Rectangle {
34+ id: background
35 anchors.fill: parent
36 color: headerStyle.backgroundColor
37+
38+ // The border is shown to warn the developer that the AppHeader
39+ // is deprecated. See bug #1583636
40+ property bool showBorder: styledItem.hasOwnProperty("showDeprecatedWarning") &&
41+ styledItem.showDeprecatedWarning
42+ border {
43+ width: background.showBorder ? units.gu(0.5) : 0
44+ color: background.showBorder ? UbuntuColors.red : "transparent"
45+ }
46 }
47
48 // FIXME: When the three panel color properties below are removed,

Subscribers

People subscribed via source and target branches