Merge lp:~diegosarmentero/ubuntuone-windows-installer/icon-signin into lp:ubuntuone-windows-installer

Proposed by Diego Sarmentero
Status: Merged
Approved by: Diego Sarmentero
Approved revision: 32
Merged at revision: 27
Proposed branch: lp:~diegosarmentero/ubuntuone-windows-installer/icon-signin
Merge into: lp:ubuntuone-windows-installer
Diff against target: 198 lines (+52/-44)
5 files modified
data/qt/choose_sign_in.ui (+25/-29)
data/qt/images.qrc (+2/-1)
data/qt/ubuntuone.qss (+7/-0)
ubuntuone_installer/gui/qt/gui.py (+0/-14)
ubuntuone_installer/gui/qt/main/windows.py (+18/-0)
To merge this branch: bzr merge lp:~diegosarmentero/ubuntuone-windows-installer/icon-signin
Reviewer Review Type Date Requested Status
Natalia Bidart (community) Approve
Roberto Alsina (community) Approve
Review via email: mp+70603@code.launchpad.net

Commit message

Moving StyleSheet and font setup to windows.py.
Adding application icon.
Fixing Sign in Page UI Style.

Description of the change

Moving StyleSheet and font setup to windows.py.
Adding application icon.
Fixing Sign in Page UI Style.

To post a comment you must log in.
27. By Diego Sarmentero

adding images and removing unnecessary xml from choose_sing_in.xml

Revision history for this message
Roberto Alsina (ralsina) wrote :

+1

review: Approve
28. By Diego Sarmentero

using QResource to load the qss file.

29. By Diego Sarmentero

removing comment lines

30. By Diego Sarmentero

merge and conflicts resolved.

31. By Diego Sarmentero

removing empty pointless line in qss

32. By Diego Sarmentero

fixing malformed qss.

Revision history for this message
Natalia Bidart (nataliabidart) wrote :

Looks good!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'data/qt/choose_sign_in.ui'
2--- data/qt/choose_sign_in.ui 2011-08-04 15:32:35 +0000
3+++ data/qt/choose_sign_in.ui 2011-08-05 19:56:37 +0000
4@@ -14,22 +14,15 @@
5 <string>WizardPage</string>
6 </property>
7 <layout class="QVBoxLayout" name="verticalLayout_2">
8+ <property name="leftMargin">
9+ <number>0</number>
10+ </property>
11+ <property name="rightMargin">
12+ <number>0</number>
13+ </property>
14 <item>
15 <layout class="QHBoxLayout" name="horizontalLayout_2">
16 <item>
17- <spacer name="horizontalSpacer_3">
18- <property name="orientation">
19- <enum>Qt::Horizontal</enum>
20- </property>
21- <property name="sizeHint" stdset="0">
22- <size>
23- <width>40</width>
24- <height>20</height>
25- </size>
26- </property>
27- </spacer>
28- </item>
29- <item>
30 <widget class="QLabel" name="image_label">
31 <property name="sizePolicy">
32 <sizepolicy hsizetype="Fixed" vsizetype="Fixed">
33@@ -50,7 +43,7 @@
34 <enum>Qt::PlainText</enum>
35 </property>
36 <property name="pixmap">
37- <pixmap resource="images.qrc">:/win_install_img_placeholder.png</pixmap>
38+ <pixmap resource="images.qrc">:/win_installer_graphic.png</pixmap>
39 </property>
40 <property name="alignment">
41 <set>Qt::AlignCenter</set>
42@@ -60,19 +53,6 @@
43 </property>
44 </widget>
45 </item>
46- <item>
47- <spacer name="horizontalSpacer_4">
48- <property name="orientation">
49- <enum>Qt::Horizontal</enum>
50- </property>
51- <property name="sizeHint" stdset="0">
52- <size>
53- <width>40</width>
54- <height>20</height>
55- </size>
56- </property>
57- </spacer>
58- </item>
59 </layout>
60 </item>
61 <item>
62@@ -85,14 +65,30 @@
63 </font>
64 </property>
65 <property name="text">
66- <string>Ubuntu One has installed!</string>
67+ <string>Congratulations, Ubuntu One is installed!</string>
68 </property>
69 <property name="alignment">
70- <set>Qt::AlignCenter</set>
71+ <set>Qt::AlignLeading|Qt::AlignLeft|Qt::AlignVCenter</set>
72 </property>
73 </widget>
74 </item>
75 <item>
76+ <spacer name="verticalSpacer_3">
77+ <property name="orientation">
78+ <enum>Qt::Vertical</enum>
79+ </property>
80+ <property name="sizeType">
81+ <enum>QSizePolicy::Fixed</enum>
82+ </property>
83+ <property name="sizeHint" stdset="0">
84+ <size>
85+ <width>20</width>
86+ <height>30</height>
87+ </size>
88+ </property>
89+ </spacer>
90+ </item>
91+ <item>
92 <layout class="QHBoxLayout" name="horizontalLayout">
93 <item>
94 <spacer name="horizontalSpacer_2">
95
96=== modified file 'data/qt/images.qrc'
97--- data/qt/images.qrc 2011-07-27 14:53:56 +0000
98+++ data/qt/images.qrc 2011-08-05 19:56:37 +0000
99@@ -13,6 +13,7 @@
100 <file>../progress_finish_orange.png</file>
101 <file>../progress_tick.png</file>
102 <file>../u1icon.png</file>
103- <file>../win_install_img_placeholder.png</file>
104+ <file>../ubuntu_one_favicon.png</file>
105+ <file>../win_installer_graphic.png</file>
106 </qresource>
107 </RCC>
108
109=== modified file 'data/qt/ubuntuone.qss'
110--- data/qt/ubuntuone.qss 2011-08-03 12:54:56 +0000
111+++ data/qt/ubuntuone.qss 2011-08-05 19:56:37 +0000
112@@ -123,6 +123,13 @@
113 font-size: 24px;
114 }
115
116+QLabel#image_label{
117+ border-style: dotted;
118+ border-color: #939389;
119+ border-bottom-width: 1px;
120+ border-top-width: 1px;
121+}
122+
123 QLabel#title_label{
124 font-size: 20px;
125 }
126
127=== added file 'data/ubuntu_one_favicon.png'
128Binary files data/ubuntu_one_favicon.png 1970-01-01 00:00:00 +0000 and data/ubuntu_one_favicon.png 2011-08-05 19:56:37 +0000 differ
129=== removed file 'data/win_install_img_placeholder.png'
130Binary files data/win_install_img_placeholder.png 2011-07-27 14:53:56 +0000 and data/win_install_img_placeholder.png 1970-01-01 00:00:00 +0000 differ
131=== added file 'data/win_installer_graphic.png'
132Binary files data/win_installer_graphic.png 1970-01-01 00:00:00 +0000 and data/win_installer_graphic.png 2011-08-05 19:56:37 +0000 differ
133=== modified file 'ubuntuone_installer/gui/qt/gui.py'
134--- ubuntuone_installer/gui/qt/gui.py 2011-08-05 19:01:03 +0000
135+++ ubuntuone_installer/gui/qt/gui.py 2011-08-05 19:56:37 +0000
136@@ -70,11 +70,6 @@
137 setup_account_ui,
138 )
139
140-# Module used to include the resources into this file
141-# pylint: disable=W0611
142-from ubuntuone_installer.gui.qt.ui import images_rc
143-# pylint: enable=W0611
144-
145 from ubuntuone_installer.gui.qt.are_you_sure import AreYouSure
146 from ubuntuone_installer.gui.qt.currentuser import CurrentUserController
147 from ubuntuone_installer.gui.qt.forgotten import ForgottenPasswordController
148@@ -398,15 +393,6 @@
149 # Set Wizard buttons style
150 self.button(QtGui.QWizard.NextButton).setDefault(True)
151
152- # Set Application Style Sheet
153- app = QtGui.QApplication.instance()
154- qss_file = QtCore.QFile(":/ubuntuone.qss")
155- qss_file.open(QtCore.QFile.ReadOnly)
156- stylesheet = QtCore.QLatin1String(qss_file.readAll())
157- app.setStyleSheet(stylesheet)
158- # Apply font to the entire application
159- QtGui.QFontDatabase.addApplicationFont(':/Ubuntu-R.ttf')
160- QtGui.QFontDatabase.addApplicationFont(':/Ubuntu-B.ttf')
161 credtool = CredentialsManagementTool()
162 d = credtool.find_credentials()
163 d.addCallback(self.creds_cb)
164
165=== modified file 'ubuntuone_installer/gui/qt/main/windows.py'
166--- ubuntuone_installer/gui/qt/main/windows.py 2011-08-02 20:05:25 +0000
167+++ ubuntuone_installer/gui/qt/main/windows.py 2011-08-05 19:56:37 +0000
168@@ -22,6 +22,13 @@
169 import sys
170
171 from PyQt4 import QtGui
172+from PyQt4 import QtCore
173+from PyQt4.Qt import qUncompress
174+
175+# Module used to include the resources into this file
176+# pylint: disable=W0611
177+from ubuntuone_installer.gui.qt.ui import images_rc
178+# pylint: enable=W0611
179
180
181 def stop(*args):
182@@ -40,5 +47,16 @@
183 from twisted.internet import reactor
184 window = MainWindow(close_callback=stop)
185 app.window = window
186+
187+ # Set Application Style Sheet
188+ app = QtGui.QApplication.instance()
189+ qss = QtCore.QResource(":/ubuntuone.qss")
190+ app.setStyleSheet(QtCore.QString(qUncompress(qss.data())))
191+ # Apply font to the entire application
192+ QtGui.QFontDatabase.addApplicationFont(':/Ubuntu-R.ttf')
193+ QtGui.QFontDatabase.addApplicationFont(':/Ubuntu-B.ttf')
194+ # Set Application Icon
195+ app.setWindowIcon(QtGui.QIcon(":/ubuntu_one_favicon.png"))
196+
197 window.show()
198 reactor.run()

Subscribers

People subscribed via source and target branches