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

Proposed by Tim Peeters on 2015-08-06
Status: Merged
Approved by: Zsombor Egri on 2015-08-11
Approved revision: 1600
Merged at revision: 1598
Proposed branch: lp:~tpeeters/ubuntu-ui-toolkit/white
Merge into: lp:ubuntu-ui-toolkit/staging
Diff against target: 181 lines (+37/-50)
7 files modified
documentation/ubuntu-theming.qdoc (+4/-4)
src/Ubuntu/Components/1.3/MainViewBase.qml (+2/-0)
src/Ubuntu/Components/Themes/Ambiance/1.3/MainViewStyle.qml (+28/-20)
src/Ubuntu/Components/Themes/Ambiance/1.3/Palette.qml (+1/-1)
src/Ubuntu/Components/Themes/SuruDark/1.3/ListItemStyle.qml (+2/-1)
src/Ubuntu/Components/Themes/SuruDark/1.3/MainViewStyle.qml (+0/-23)
src/Ubuntu/Components/Themes/SuruDark/SuruDark.pro (+0/-1)
To merge this branch: bzr merge lp:~tpeeters/ubuntu-ui-toolkit/white
Reviewer Review Type Date Requested Status
Zsombor Egri (community) 2015-08-06 Approve on 2015-08-11
PS Jenkins bot continuous-integration Approve on 2015-08-11
Jouni Helminen (community) Approve on 2015-08-10
Review via email: mp+267270@code.launchpad.net

Commit Message

Make the default MainView background plain white.

To post a comment you must log in.
lp:~tpeeters/ubuntu-ui-toolkit/white updated on 2015-08-06
1597. By Tim Peeters on 2015-08-06

clean

lp:~tpeeters/ubuntu-ui-toolkit/white updated on 2015-08-10
1598. By Tim Peeters on 2015-08-10

link bug

Tim Peeters (tpeeters) wrote :

Screenshots attached to the bug report.

Jouni Helminen (jounihelminen) wrote :

looks ok

review: Approve
lp:~tpeeters/ubuntu-ui-toolkit/white updated on 2015-08-11
1599. By Tim Peeters on 2015-08-11

sync staging

1600. By Tim Peeters on 2015-08-11

fix docs

Zsombor Egri (zsombi) wrote :

I guess we gained some speed on MainView creation time now.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'documentation/images/surudark-theme-13.png'
2Binary files documentation/images/surudark-theme-13.png 2015-04-25 07:10:57 +0000 and documentation/images/surudark-theme-13.png 2015-08-11 10:01:50 +0000 differ
3=== modified file 'documentation/ubuntu-theming.qdoc'
4--- documentation/ubuntu-theming.qdoc 2015-06-01 15:35:58 +0000
5+++ documentation/ubuntu-theming.qdoc 2015-08-11 10:01:50 +0000
6@@ -270,11 +270,11 @@
7 * The extended (or even overridden) style component documents must follow the naming
8 * convention, and must have the names implied by the components styled.
9 *
10- * SuruDark theme overrides the Ambiance theme's MainViewStyle, therefore declares the
11- * MainViewStyle.qml document, with the following content:
12- * \snippet Themes/SuruDark/1.3/MainViewStyle.qml 0
13+ * SuruDark theme overrides the Ambiance theme's ListItemStyle, therefore declares the
14+ * ListItemStyle.qml document, with the following content:
15+ * \snippet Themes/SuruDark/1.3/ListItemStyle.qml 0
16 *
17- * The style imports the Ambiance theme module, and extends the Ambiance MainViewStyle
18+ * The style imports the Ambiance theme module, and extends the Ambiance ListItemStyle
19 * component. The same is done in the other style components. However, Palette defines
20 * own values, and does not re-use Ambiance palette values.
21 * \snippet Themes/SuruDark/1.3/Palette.qml 0
22
23=== modified file 'src/Ubuntu/Components/1.3/MainViewBase.qml'
24--- src/Ubuntu/Components/1.3/MainViewBase.qml 2015-07-22 14:35:48 +0000
25+++ src/Ubuntu/Components/1.3/MainViewBase.qml 2015-08-11 10:01:50 +0000
26@@ -99,6 +99,8 @@
27 // theme is inherited from PageTreeNode, no need to update versioning
28 styleName: "MainViewStyle"
29
30+ // FIXME: Define the colors in MainViewStyle and get rid of the properties
31+ // in MainViewBase.
32 property color headerColor: backgroundColor
33 property color backgroundColor: theme.palette.normal.background
34 property color footerColor: backgroundColor
35
36=== modified file 'src/Ubuntu/Components/Themes/Ambiance/1.3/MainViewStyle.qml'
37--- src/Ubuntu/Components/Themes/Ambiance/1.3/MainViewStyle.qml 2015-04-25 07:36:13 +0000
38+++ src/Ubuntu/Components/Themes/Ambiance/1.3/MainViewStyle.qml 2015-08-11 10:01:50 +0000
39@@ -17,43 +17,51 @@
40 import Ubuntu.Components 1.3
41
42 Item {
43- // styling properties
44 anchors.fill: parent
45 z: -1
46 id: mainViewStyle
47
48 /*!
49- The background texture of the main view. The image will be drawn over the background color,
50- so if it has (semi-)transparent pixels, in those pixels the background color will be visible.
51- */
52- property url backgroundSource: Qt.resolvedUrl("../artwork/background_paper.png")
53+ Color of the header's background.
54+
55+ \sa backgroundColor, footerColor
56+ */
57+ property color headerColor: styledItem.headerColor
58+
59+ /*!
60+ Color of the background.
61+
62+ The background is usually a single color. However if \l headerColor
63+ or \l footerColor are set then a gradient of colors will be drawn.
64+
65+ \sa footerColor, headerColor
66+ */
67+ property color backgroundColor: styledItem.backgroundColor
68+
69+ /*!
70+ Color of the footer's background.
71+
72+ \sa backgroundColor, headerColor
73+ */
74+ property color footerColor: styledItem.footerColor
75
76 Gradient {
77 id: backgroundGradient
78- GradientStop { position: 0.0; color: styledItem.headerColor }
79- GradientStop { position: 0.83; color: styledItem.backgroundColor }
80- GradientStop { position: 1.0; color: styledItem.footerColor }
81+ GradientStop { position: 0.0; color: mainViewStyle.headerColor }
82+ GradientStop { position: 0.83; color: mainViewStyle.backgroundColor }
83+ GradientStop { position: 1.0; color: mainViewStyle.footerColor }
84 }
85
86 Rectangle {
87 id: backgroundColor
88 anchors.fill: parent
89- color: styledItem.backgroundColor
90+ color: mainViewStyle.backgroundColor
91 gradient: internals.isGradient ? backgroundGradient : null
92 }
93
94- Image {
95- id: backgroundTexture
96- anchors.fill: parent
97- source: mainViewStyle.backgroundSource
98- fillMode: Image.Tile
99- asynchronous: true
100- cache: false
101- }
102-
103 QtObject {
104 id: internals
105- property bool isGradient: styledItem.backgroundColor != styledItem.headerColor ||
106- styledItem.backgroundColor != styledItem.footerColor
107+ property bool isGradient: mainViewStyle.backgroundColor != mainViewStyle.headerColor ||
108+ mainViewStyle.backgroundColor != mainViewStyle.footerColor
109 }
110 }
111
112=== modified file 'src/Ubuntu/Components/Themes/Ambiance/1.3/Palette.qml'
113--- src/Ubuntu/Components/Themes/Ambiance/1.3/Palette.qml 2015-04-27 07:19:22 +0000
114+++ src/Ubuntu/Components/Themes/Ambiance/1.3/Palette.qml 2015-08-11 10:01:50 +0000
115@@ -20,7 +20,7 @@
116
117 Palette {
118 normal {
119- background: "#EDEDED"
120+ background: "#FFFFFF"
121 backgroundText: "#81888888"
122 base: Qt.rgba(0, 0, 0, 0.1)
123 baseText: UbuntuColors.lightGrey
124
125=== modified file 'src/Ubuntu/Components/Themes/SuruDark/1.3/ListItemStyle.qml'
126--- src/Ubuntu/Components/Themes/SuruDark/1.3/ListItemStyle.qml 2015-07-13 22:16:37 +0000
127+++ src/Ubuntu/Components/Themes/SuruDark/1.3/ListItemStyle.qml 2015-08-11 10:01:50 +0000
128@@ -13,7 +13,7 @@
129 * You should have received a copy of the GNU Lesser General Public License
130 * along with this program. If not, see <http://www.gnu.org/licenses/>.
131 */
132-
133+//![0]
134 import QtQuick 2.4
135 import Ubuntu.Components.Themes.Ambiance 1.3 as Ambiance
136
137@@ -23,3 +23,4 @@
138 // 40% lighter than foreground color
139 trailingForegroundColor: Qt.lighter(theme.palette.normal.foreground, 1.4)
140 }
141+//![0]
142
143=== removed file 'src/Ubuntu/Components/Themes/SuruDark/1.3/MainViewStyle.qml'
144--- src/Ubuntu/Components/Themes/SuruDark/1.3/MainViewStyle.qml 2015-04-24 14:12:12 +0000
145+++ src/Ubuntu/Components/Themes/SuruDark/1.3/MainViewStyle.qml 1970-01-01 00:00:00 +0000
146@@ -1,23 +0,0 @@
147-/*
148- * Copyright 2015 Canonical Ltd.
149- *
150- * This program is free software; you can redistribute it and/or modify
151- * it under the terms of the GNU Lesser General Public License as published by
152- * the Free Software Foundation; version 3.
153- *
154- * This program is distributed in the hope that it will be useful,
155- * but WITHOUT ANY WARRANTY; without even the implied warranty of
156- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
157- * GNU Lesser General Public License for more details.
158- *
159- * You should have received a copy of the GNU Lesser General Public License
160- * along with this program. If not, see <http://www.gnu.org/licenses/>.
161- */
162-//![0]
163-import QtQuick 2.4
164-import Ubuntu.Components.Themes.Ambiance 1.3
165-
166-MainViewStyle {
167- backgroundSource: ""
168-}
169-//![0]
170
171=== modified file 'src/Ubuntu/Components/Themes/SuruDark/SuruDark.pro'
172--- src/Ubuntu/Components/Themes/SuruDark/SuruDark.pro 2015-07-24 08:02:46 +0000
173+++ src/Ubuntu/Components/Themes/SuruDark/SuruDark.pro 2015-08-11 10:01:50 +0000
174@@ -8,7 +8,6 @@
175 1.2/Palette.qml \
176 1.2/TabBarStyle.qml \
177 1.2/ListItemStyle.qml \
178- 1.3/MainViewStyle.qml \
179 1.3/OptionSelectorStyle.qml \
180 1.3/Palette.qml \
181 1.3/TabBarStyle.qml \

Subscribers

People subscribed via source and target branches