Merge lp:~agateau/ubiquity/kde-fix-broken-sidebar-layout-on-greeter into lp:ubiquity

Proposed by Aurélien Gâteau
Status: Merged
Merged at revision: 6192
Proposed branch: lp:~agateau/ubiquity/kde-fix-broken-sidebar-layout-on-greeter
Merge into: lp:ubiquity
Diff against target: 102 lines (+22/-17)
4 files modified
gui/qt/app.ui (+14/-11)
gui/qt/breadcrumb_current.qss (+1/-2)
ubiquity/frontend/kde_components/Breadcrumb.py (+6/-2)
ubiquity/frontend/kde_ui.py (+1/-2)
To merge this branch: bzr merge lp:~agateau/ubiquity/kde-fix-broken-sidebar-layout-on-greeter
Reviewer Review Type Date Requested Status
Jonathan Riddell Pending
Review via email: mp+216365@code.launchpad.net
To post a comment you must log in.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'gui/qt/app.ui'
--- gui/qt/app.ui 2014-04-16 09:52:59 +0000
+++ gui/qt/app.ui 2014-04-17 16:18:12 +0000
@@ -184,7 +184,7 @@
184 <item>184 <item>
185 <widget class="QWidget" name="steps_widget" native="true">185 <widget class="QWidget" name="steps_widget" native="true">
186 <property name="sizePolicy">186 <property name="sizePolicy">
187 <sizepolicy hsizetype="Minimum" vsizetype="Preferred">187 <sizepolicy hsizetype="Minimum" vsizetype="Minimum">
188 <horstretch>0</horstretch>188 <horstretch>0</horstretch>
189 <verstretch>0</verstretch>189 <verstretch>0</verstretch>
190 </sizepolicy>190 </sizepolicy>
@@ -196,20 +196,23 @@
196 <property name="margin">196 <property name="margin">
197 <number>0</number>197 <number>0</number>
198 </property>198 </property>
199 <item>
200 <widget class="QWidget" name="steps_filler_widget" native="true">
201 <property name="sizePolicy">
202 <sizepolicy hsizetype="Preferred" vsizetype="Expanding">
203 <horstretch>0</horstretch>
204 <verstretch>0</verstretch>
205 </sizepolicy>
206 </property>
207 </widget>
208 </item>
209 </layout>199 </layout>
210 </widget>200 </widget>
211 </item>201 </item>
212 <item>202 <item>
203 <spacer name="verticalSpacer">
204 <property name="orientation">
205 <enum>Qt::Vertical</enum>
206 </property>
207 <property name="sizeHint" stdset="0">
208 <size>
209 <width>40</width>
210 <height>20</height>
211 </size>
212 </property>
213 </spacer>
214 </item>
215 <item>
213 <widget class="QFrame" name="sidebar_footer_frame">216 <widget class="QFrame" name="sidebar_footer_frame">
214 <property name="sizePolicy">217 <property name="sizePolicy">
215 <sizepolicy hsizetype="Preferred" vsizetype="Minimum">218 <sizepolicy hsizetype="Preferred" vsizetype="Minimum">
216219
=== modified file 'gui/qt/breadcrumb_current.qss'
--- gui/qt/breadcrumb_current.qss 2013-02-28 06:42:46 +0000
+++ gui/qt/breadcrumb_current.qss 2014-04-17 16:18:12 +0000
@@ -1,4 +1,4 @@
1QFrame {1QFrame[isBreadcrumb=true] {
2 border-top-width: 10px;2 border-top-width: 10px;
3 border-bottom-width: 10px;3 border-bottom-width: 10px;
4 background-image: none;4 background-image: none;
@@ -7,7 +7,6 @@
77
8.QLabel {8.QLabel {
9 color: #333;9 color: #333;
10 border-width: 0px;
11 border-image: none;10 border-image: none;
12}11}
1312
1413
=== modified file 'ubiquity/frontend/kde_components/Breadcrumb.py'
--- ubiquity/frontend/kde_components/Breadcrumb.py 2013-03-07 18:24:39 +0000
+++ ubiquity/frontend/kde_components/Breadcrumb.py 2014-04-17 16:18:12 +0000
@@ -30,13 +30,17 @@
30__all__ = ["Breadcrumb"]30__all__ = ["Breadcrumb"]
3131
3232
33class Breadcrumb(QtGui.QFrame):33# HACK! Breadcrumb should not have to inherit from QLabel! Inheriting from
34# QFrame should be enough, but right now (Trusty) if I change Breadcrumb to
35# inherit from QFrame then _mainLabel is not given enough vertical space and
36# its text gets cropped at the bottom.
37class Breadcrumb(QtGui.QLabel):
34 TODO = 038 TODO = 0
35 CURRENT = 139 CURRENT = 1
36 DONE = 240 DONE = 2
3741
38 def __init__(self, parent=None):42 def __init__(self, parent=None):
39 QtGui.QFrame.__init__(self, parent)43 super(Breadcrumb, self).__init__(parent)
40 self.setProperty("isBreadcrumb", True)44 self.setProperty("isBreadcrumb", True)
4145
42 self._tickLabel = QtGui.QLabel()46 self._tickLabel = QtGui.QLabel()
4347
=== modified file 'ubiquity/frontend/kde_ui.py'
--- ubiquity/frontend/kde_ui.py 2014-03-06 06:42:00 +0000
+++ ubiquity/frontend/kde_ui.py 2014-04-17 16:18:12 +0000
@@ -315,8 +315,7 @@
315 widget = Breadcrumb()315 widget = Breadcrumb()
316 widget.setObjectName(name)316 widget.setObjectName(name)
317 layout = self.ui.steps_widget.layout()317 layout = self.ui.steps_widget.layout()
318 # "- 1" to insert before the bottom spacer318 layout.addWidget(widget)
319 layout.insertWidget(layout.count() - 1, widget)
320 return widget319 return widget
321320
322 def _apply_stylesheet(self):321 def _apply_stylesheet(self):

Subscribers

People subscribed via source and target branches

to status/vote changes: