Merge lp:~ralsina/ubuntuone-control-panel/u1cp-windows-styling into lp:ubuntuone-control-panel

Proposed by Roberto Alsina
Status: Rejected
Rejected by: Natalia Bidart
Proposed branch: lp:~ralsina/ubuntuone-control-panel/u1cp-windows-styling
Merge into: lp:ubuntuone-control-panel
Diff against target: 210 lines (+109/-48)
3 files modified
data/qt/linux.qss (+60/-0)
data/qt/ubuntuone.qss (+0/-48)
data/qt/windows.qss (+49/-0)
To merge this branch: bzr merge lp:~ralsina/ubuntuone-control-panel/u1cp-windows-styling
Reviewer Review Type Date Requested Status
Natalia Bidart (community) Needs Resubmitting
Review via email: mp+98704@code.launchpad.net

Description of the change

Disable the focus hacks on windows, where they are not needed and have bad side-effects.

To post a comment you must log in.
296. By Roberto Alsina

solved conflicts

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

Proposed https://code.launchpad.net/~nataliabidart/ubuntuone-control-panel/u1cp-windows-styling/+merge/99525 to solve some code duplication from this branch (and because Roberto is in a meeting and we need this ASAP).

review: Needs Resubmitting

Unmerged revisions

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'data/qt/linux.qss'
2--- data/qt/linux.qss 2012-03-16 21:10:54 +0000
3+++ data/qt/linux.qss 2012-03-23 19:21:19 +0000
4@@ -1,2 +1,62 @@
5 /* Styles specific to the linux platform */
6
7+QPushButton:focus {
8+ border-width: 2px;
9+ /* reduce the padding since we have a 2px border now */
10+ padding: 5px;
11+ padding-left: 19px;
12+ padding-right: 19px;
13+ /* hack to make the mild-orange focused box dissapear */
14+ padding-top: 100%;
15+ padding-bottom: 100%;
16+ /* end of hack */
17+}
18+
19+GoToWebButton#help_button:focus {
20+ border: 2px solid #dd4814;
21+ /* hack to make the mild-orange focused box dissapear */
22+ padding-top: 100%;
23+ padding-bottom: 100%;
24+ /* end of hack */
25+ /* Compensate for border so text doesn't move */
26+ padding-left: 0px;
27+ padding-right: 13px;
28+}
29+
30+GoToWebButton#share_publish_button:focus {
31+ border: 2px solid #aea79f;
32+ /* hack to make the mild-orange focused box dissapear */
33+ padding-top: 100%;
34+ padding-bottom: 100%;
35+ /* end of hack */
36+ /* Compensate for border so text doesn't move */
37+ padding-left: 8px;
38+ padding-right: 23px;
39+}
40+
41+QTabBar::tab:selected {
42+ background-color: white;
43+ border-top-left-radius: 5px;
44+ border-top-right-radius: 5px;
45+ border-style: solid;
46+ padding: 12px;
47+ border-top-color: #939389;
48+ border-right-color: #939389;
49+ border-left-color: white;
50+ border-bottom-color: white;
51+ border-width: 1px;
52+}
53+
54+QTabBar::tab:hover,
55+QTabBar::tab:focus {
56+ text-decoration: underline;
57+ /* hack to make the mild-orange focused box dissapear */
58+ padding-left: 1000px;
59+ padding-right: 1000px;
60+ /* end of hack */
61+}
62+
63+QPushButton#twitter_button:focus,
64+QPushButton#facebook_button:focus {
65+ border: 2px solid #dd4814;
66+}
67
68=== modified file 'data/qt/ubuntuone.qss'
69--- data/qt/ubuntuone.qss 2012-03-22 20:28:48 +0000
70+++ data/qt/ubuntuone.qss 2012-03-23 19:21:19 +0000
71@@ -75,18 +75,6 @@
72 border-width: 1px;
73 }
74
75-QPushButton:focus {
76- border-width: 2px;
77- /* reduce the padding since we have a 2px border now */
78- padding: 5px;
79- padding-left: 19px;
80- padding-right: 19px;
81- /* hack to make the mild-orange focused box dissapear */
82- padding-top: 100%;
83- padding-bottom: 100%;
84- /* end of hack */
85-}
86-
87 QPushButton:disabled {
88 background: qlineargradient(x1: 0, y1: 0, x2: 0, y2: 1,
89 stop: 0 #eaeaea, stop: 1.0 #cacaca);
90@@ -156,11 +144,6 @@
91 border: none;
92 }
93
94-QPushButton#twitter_button:focus,
95-QPushButton#facebook_button:focus {
96- border: 2px solid #dd4814;
97-}
98-
99 GoToWebButton#help_button {
100 background: transparent;
101 border: none;
102@@ -173,17 +156,6 @@
103 background-origin: margin;
104 }
105
106-GoToWebButton#help_button:focus {
107- border: 2px solid #dd4814;
108- /* hack to make the mild-orange focused box dissapear */
109- padding-top: 100%;
110- padding-bottom: 100%;
111- /* end of hack */
112- /* Compensate for border so text doesn't move */
113- padding-left: 0px;
114- padding-right: 13px;
115-}
116-
117 GoToWebButton#share_publish_button {
118 background: transparent;
119 border: none;
120@@ -197,17 +169,6 @@
121 background-origin: margin;
122 }
123
124-GoToWebButton#share_publish_button:focus {
125- border: 2px solid #aea79f;
126- /* hack to make the mild-orange focused box dissapear */
127- padding-top: 100%;
128- padding-bottom: 100%;
129- /* end of hack */
130- /* Compensate for border so text doesn't move */
131- padding-left: 8px;
132- padding-right: 23px;
133-}
134-
135 GoToWebButton#edit_profile_button,
136 GoToWebButton#edit_services_button,
137 GoToWebButton#get_more_space_button {
138@@ -266,15 +227,6 @@
139 border-bottom-left-radius: 0px;
140 }
141
142-QTabBar::tab:hover,
143-QTabBar::tab:focus {
144- text-decoration: underline;
145- /* hack to make the mild-orange focused box dissapear */
146- padding-left: 1000px;
147- padding-right: 1000px;
148- /* end of hack */
149-}
150-
151 QGroupBox {
152 padding-top: 30px;
153 border: none;
154
155=== modified file 'data/qt/windows.qss'
156--- data/qt/windows.qss 2012-03-15 01:25:08 +0000
157+++ data/qt/windows.qss 2012-03-23 19:21:19 +0000
158@@ -3,3 +3,52 @@
159 QWidget {
160 font-family: "Ubuntu";
161 }
162+
163+
164+QPushButton:focus {
165+ border-width: 2px;
166+ /* reduce the padding since we have a 2px border now */
167+ padding: 5px;
168+ padding-left: 19px;
169+ padding-right: 19px;
170+}
171+
172+GoToWebButton#help_button:focus {
173+ border: 2px solid #dd4814;
174+ /* Compensate for border so text doesn't move */
175+ padding-left: 0px;
176+ padding-right: 13px;
177+}
178+
179+GoToWebButton#share_publish_button:focus {
180+ border: 2px solid #aea79f;
181+ /* Compensate for border so text doesn't move */
182+ padding-left: 8px;
183+ padding-right: 23px;
184+}
185+
186+QTabBar::tab:selected {
187+ background-color: white;
188+ border-top-left-radius: 5px;
189+ border-top-right-radius: 5px;
190+ border-style: solid;
191+ padding: 12px;
192+ border-top-color: #939389;
193+ border-right-color: #939389;
194+ border-left-color: white;
195+ border-bottom-color: white;
196+ border-width: 1px;
197+}
198+
199+QTabBar::tab:hover,
200+QTabBar::tab:focus {
201+ text-decoration: underline;
202+ padding-left: 0px;
203+ padding-right: 0px;
204+}
205+
206+QPushButton#twitter_button:focus,
207+QPushButton#facebook_button:focus {
208+ border: 2px solid #dd4814;
209+ padding: 0px;
210+}

Subscribers

People subscribed via source and target branches