Merge lp:~whosdaz/ubuntu-html5-theme/googlecodein into lp:ubuntu-html5-theme

Proposed by Matthew Allen
Status: Superseded
Proposed branch: lp:~whosdaz/ubuntu-html5-theme/googlecodein
Merge into: lp:ubuntu-html5-theme
Diff against target: 551 lines (+94/-105)
4 files modified
0.1/ambiance/css/appTemplate.css (+48/-65)
0.1/ambiance/css/sass/default.scss (+3/-1)
0.1/ambiance/css/sass/toggles.scss (+37/-34)
examples/html5-theme/widgets/Toggles.html (+6/-5)
To merge this branch: bzr merge lp:~whosdaz/ubuntu-html5-theme/googlecodein
Reviewer Review Type Date Requested Status
Alexandre Abreu Needs Fixing
Review via email: mp+280292@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Alexandre Abreu (abreu-alexandre) wrote :

Thank you for working on that ! :)

A few comments:

- the checkboxes appear to be slightly bigger than the qml version,
- for checkboxes, the tick images are not properly aligned in the box, and they are a bit smaller than the qml version,
- the back color should not be orange but green,
- the switch ticks are a bit bolder than the checkbox versions,
- the back color for the uncheck is not the right grey,

review: Needs Fixing
218. By Matthew Allen

Fixed background colour of button, fixed Orange accents to Green, fixed sizing of tickboxes

Unmerged revisions

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file '0.1/ambiance/css/appTemplate.css'
2--- 0.1/ambiance/css/appTemplate.css 2015-01-29 15:14:59 +0000
3+++ 0.1/ambiance/css/appTemplate.css 2015-12-12 11:57:23 +0000
4@@ -672,7 +672,7 @@
5 }
6
7 .textfield.clear:focus {
8- background: url(artwork/icon_clear@20.png) no-repeat 97% center white;
9+ background: url(artwork/icon_clear@20.png) no-repeat 97% center #fff;
10 background-size: 12%;
11 }
12
13@@ -785,7 +785,6 @@
14 from {
15 -webkit-transform: rotate(1deg);
16 }
17-
18 to {
19 -webkit-transform: rotate(360deg);
20 }
21@@ -794,11 +793,9 @@
22 0% {
23 opacity: 0;
24 }
25-
26 50% {
27 opacity: 1;
28 }
29-
30 100% {
31 opacity: 0;
32 }
33@@ -928,43 +925,33 @@
34 0% {
35 -webkit-transform: translate3d(0, 0, 0);
36 }
37-
38 10% {
39 -webkit-transform: translate3d(-20px, 0, 0);
40 }
41-
42 20% {
43 -webkit-transform: translate3d(20px, 0, 0);
44 }
45-
46 30% {
47 -webkit-transform: translate3d(-20px, 0, 0);
48 }
49-
50 40% {
51 -webkit-transform: translate3d(20px, 0, 0);
52 }
53-
54 50% {
55 -webkit-transform: translate3d(-20px, 0, 0);
56 }
57-
58 60% {
59 -webkit-transform: translate3d(20px, 0, 0);
60 }
61-
62 70% {
63 -webkit-transform: translate3d(-20px, 0, 0);
64 }
65-
66 80% {
67 -webkit-transform: translate3d(20px, 0, 0);
68 }
69-
70 90% {
71 -webkit-transform: translate3d(-20px, 0, 0);
72 }
73-
74 100% {
75 -webkit-transform: translate3d(0, 0, 0);
76 }
77@@ -973,43 +960,33 @@
78 0% {
79 -webkit-transform: translate3d(0, 0, 0);
80 }
81-
82 10% {
83 -webkit-transform: translate3d(-20px, 0, 0);
84 }
85-
86 20% {
87 -webkit-transform: translate3d(20px, 0, 0);
88 }
89-
90 30% {
91 -webkit-transform: translate3d(-20px, 0, 0);
92 }
93-
94 40% {
95 -webkit-transform: translate3d(20px, 0, 0);
96 }
97-
98 50% {
99 -webkit-transform: translate3d(-20px, 0, 0);
100 }
101-
102 60% {
103 -webkit-transform: translate3d(20px, 0, 0);
104 }
105-
106 70% {
107 -webkit-transform: translate3d(-20px, 0, 0);
108 }
109-
110 80% {
111 -webkit-transform: translate3d(20px, 0, 0);
112 }
113-
114 90% {
115 -webkit-transform: translate3d(-20px, 0, 0);
116 }
117-
118 100% {
119 -webkit-transform: translate3d(0, 0, 0);
120 }
121@@ -1056,7 +1033,7 @@
122 background: #dd4814;
123 border: 2px solid #dd4814;
124 border-radius: 6px;
125- color: white;
126+ color: #ffffff;
127 font-family: "Ubuntu";
128 font-size: 1rem;
129 padding: 9px 25px;
130@@ -1066,21 +1043,21 @@
131 }
132 [data-role="button"].positive {
133 background: #38b44a;
134- color: white;
135+ color: #ffffff;
136 border-color: #38b44a;
137 }
138 [data-role="button"].information {
139 background: #2ab7ec;
140- color: white;
141+ color: #ffffff;
142 border-color: #2ab7ec;
143 }
144 [data-role="button"].secondary {
145- background: white;
146+ background: #ffffff;
147 color: #888888;
148 }
149 [data-role="button"].negative {
150 background: #929292;
151- color: white;
152+ color: #ffffff;
153 border-color: #929292;
154 }
155
156@@ -1205,14 +1182,16 @@
157 * You should have received a copy of the GNU Lesser General Public
158 * License along with this program. If not, see
159 * <http://www.gnu.org/licenses/>.
160+ * BOXES SCALED BY A FACTOR OF 0.65
161 */
162 label:not([for]) {
163 display: inline-block;
164 vertical-align: middle;
165- width: 4.5rem;
166+ width: 2.925rem;
167 height: 2.2rem;
168 position: relative;
169 background: none;
170+ margin-top: 15px;
171 }
172 label:not([for]) input[type="checkbox"] {
173 margin: 0;
174@@ -1230,92 +1209,92 @@
175 left: 50%;
176 top: 50%;
177 margin: -1.1rem 0 0 -1.1rem;
178- width: 2rem;
179- height: 2rem;
180+ width: 1.25rem;
181+ height: 1.15rem;
182 pointer-events: none;
183 border-radius: 6px;
184- background: rgba(0, 0, 0, 0.4);
185- background-position-y: 30px;
186+ background: transparent;
187+ background-position-y: 5px;
188 -webkit-transition: background-image 0.3s ease-in-out, background-color 0.3s ease-in-out, background-position 0.3s ease-in-out;
189 transition: background-image 0.3s ease-in-out, background-color 0.3s ease-in-out, background-position 0.3s ease-in-out;
190 }
191 label:not([for]) input[type="checkbox"]:checked + span {
192- background: #dd4814 url(../img/tick-white@30.png) no-repeat 0.3rem 0.3rem;
193- background-size: 75% 75%;
194+ background: #00a132 url(../img/tick-white@30.png) no-repeat 0.2rem 0.2rem;
195+ background-size: 70% 70%;
196 background-position-y: 5px;
197 -webkit-transition: background-image 0.3s ease-in-out, background-color 0.3s ease-in-out, background-position 0.3s ease-in-out;
198 transition: background-image 0.3s ease-in-out, background-color 0.3s ease-in-out, background-position 0.3s ease-in-out;
199 }
200 label:not([for]) input[type="checkbox"]:disabled + span {
201- background: rgba(0, 0, 0, 0.25) !important;
202+ background: rgba(0, 0, 0, 0.025) !important;
203 }
204 label:not([for]) input[type="checkbox"]:disabled:checked + span {
205- background: rgba(221, 72, 20, 0.5) url(../img/tick-white@30.png) no-repeat 0.3rem 0.3rem !important;
206- background-size: 75% 75% !important;
207+ background: rgba(0, 161, 50, 0.5) url(../img/tick-white@30.png) no-repeat 0.2em 0.2rem !important;
208+ background-size: 70% 70% !important;
209 }
210
211 label input[type="checkbox"][data-type="switch"] + span.toggle {
212 position: absolute;
213 left: 50%;
214 top: 50%;
215- margin: -1.1rem 0 0 -2.2rem;
216- width: 4.5rem;
217- height: 2.2rem;
218+ margin: -1.1rem 0 0 -1.2rem;
219+ width: 2.925rem;
220+ height: 1.43rem;
221 pointer-events: none;
222 overflow: hidden;
223- background: rgba(136, 136, 136, 0.3);
224+ background: rgba(136, 136, 136, 0.2);
225 }
226 label input[type="checkbox"][data-type="switch"] + span.toggle > span.toggle-bg {
227- background: transparent url(../img/unCheckbox.png) no-repeat 0.4rem 0.35rem;
228- background-size: 90% 75%;
229+ background: transparent url(../img/unCheckbox.png) no-repeat 0.26rem 0.21rem;
230+ background-size: 90% 80%;
231 position: absolute;
232 top: 0;
233 left: 0;
234 z-index: 10;
235- width: 4.5rem;
236- height: 2.1rem;
237+ width: 2.925rem;
238+ height: 1.365rem;
239 -webkit-transition: background-image 0.3s ease-in-out;
240 transition: background-image 0.3s ease-in-out;
241 }
242 label input[type="checkbox"][data-type="switch"] + span.toggle > span.toggle-handle {
243 -webkit-box-sizing: border-box;
244 box-sizing: border-box;
245- -webkit-box-shadow: inset 0 1px 1px rgba(0, 0, 0, 0.3);
246- box-shadow: inset 0 1px 1px rgba(0, 0, 0, 0.3);
247+ -webkit-box-shadow: inset 0 1px 1px rgba(0, 0, 0, 0.2);
248+ box-shadow: inset 0 1px 1px rgba(0, 0, 0, 0.2);
249 position: absolute;
250- top: 2px;
251- left: 2px;
252+ top: 3px;
253+ left: 3px;
254 z-index: 2;
255- width: 2rem;
256- height: 2rem;
257+ width: 1.1rem;
258+ height: 1.1rem;
259 border-radius: 6px;
260- background: rgba(0, 0, 0, 0.4);
261+ background: rgba(0, 0, 0, 0.2);
262 -webkit-transition: background 0.3s ease-in-out, -webkit-transform 0.3s ease-in-out;
263 transition: background 0.3s ease-in-out, transform 0.3s ease-in-out;
264 }
265 label input[type="checkbox"][data-type="switch"]:checked + span.toggle {
266- background: rgba(136, 136, 136, 0.3) !important;
267+ background: rgba(136, 136, 136, 0.2) !important;
268 }
269 label input[type="checkbox"][data-type="switch"]:checked + span.toggle > span.toggle-bg {
270- background: transparent url(../img/Checkbox.png) no-repeat 0.4rem 0.35rem;
271- background-size: 90% 75%;
272+ background: transparent url(../img/Checkbox.png) no-repeat 0.26rem 0.2275rem;
273+ background-size: 90% 80%;
274 }
275 label input[type="checkbox"][data-type="switch"]:checked + span.toggle > span.toggle-handle {
276- background: #dd4814;
277- -webkit-transform: translate3d(38px, 0, 0);
278- transform: translate3d(38px, 0, 0);
279+ background: #00a132;
280+ -webkit-transform: translate3d(24.7px, 0, 0);
281+ transform: translate3d(24.7px, 0, 0);
282 }
283
284 label input[data-type="switch"]:disabled + span.toggle {
285- background: rgba(136, 136, 136, 0.3) !important;
286+ background: rgba(136, 136, 136, 0.03) !important;
287 }
288 label input[data-type="switch"]:disabled + span.toggle > span.toggle-handle {
289 opacity: 0.4;
290 }
291 label input[data-type="switch"]:disabled:checked + span.toggle > span.toggle-handle {
292 opacity: 0.4;
293- -webkit-transform: translate3d(38px, 0, 0);
294- transform: translate3d(38px, 0, 0);
295+ -webkit-transform: translate3d(24.7px, 0, 0);
296+ transform: translate3d(24.7px, 0, 0);
297 }
298
299 /*
300@@ -1862,8 +1841,8 @@
301 * <http://www.gnu.org/licenses/>.
302 */
303 [data-role="option-selector"] {
304- -webkit-box-shadow: inset 0 2px 2px 0 #d5d5d5;
305- box-shadow: inset 0 2px 2px 0 #d5d5d5;
306+ -webkit-box-shadow: inset 0 2px 2px 0 #D5D5D5;
307+ box-shadow: inset 0 2px 2px 0 #D5D5D5;
308 width: 100%;
309 height: auto;
310 margin: 0 auto;
311@@ -2043,6 +2022,8 @@
312 }
313
314 .medium-font {
315+ width: 100px;
316+ display: inline-block;
317 font-size: 14px;
318 line-height: 1.3;
319 }
320@@ -2061,3 +2042,5 @@
321 font-size: 8px;
322 line-height: 1.3;
323 }
324+
325+/*# sourceMappingURL=appTemplate.css.map */
326
327=== modified file '0.1/ambiance/css/sass/default.scss'
328--- 0.1/ambiance/css/sass/default.scss 2014-08-29 00:33:51 +0000
329+++ 0.1/ambiance/css/sass/default.scss 2015-12-12 11:57:23 +0000
330@@ -120,6 +120,8 @@
331 }
332
333 .medium-font {
334+ width: 100px;
335+ display: inline-block;
336 font-size: 14px;
337 line-height: 1.3;
338 }
339@@ -137,4 +139,4 @@
340 .xxsmall-font {
341 font-size: 8px;
342 line-height: 1.3;
343-}
344\ No newline at end of file
345+}
346
347=== modified file '0.1/ambiance/css/sass/toggles.scss'
348--- 0.1/ambiance/css/sass/toggles.scss 2014-02-19 22:25:29 +0000
349+++ 0.1/ambiance/css/sass/toggles.scss 2015-12-12 11:57:23 +0000
350@@ -18,15 +18,17 @@
351 * You should have received a copy of the GNU Lesser General Public
352 * License along with this program. If not, see
353 * <http://www.gnu.org/licenses/>.
354+ * BOXES SCALED BY A FACTOR OF 0.65
355 */
356
357 label:not([for]) {
358 display: inline-block;
359 vertical-align: middle;
360- width: 4.5rem;
361+ width: 2.925rem;
362 height: 2.2rem;
363 position: relative;
364 background: none;
365+ margin-top: 15px;
366
367 input[type="checkbox"] {
368 margin: 0;
369@@ -35,6 +37,7 @@
370 top: 0;
371 left: 0;
372
373+
374 & + span {
375 @include box_shadow (inset 0 1px 1px rgba(0, 0, 0, 0.3));
376 @include box_sizing (border-box);
377@@ -42,20 +45,20 @@
378 left: 50%;
379 top: 50%;
380 margin: -1.1rem 0 0 -1.1rem;
381- width: 2rem;
382- height: 2rem;
383+ width: 1.25rem;
384+ height: 1.15rem;
385 pointer-events: none;
386 border-radius: 6px;
387- background: rgba(0, 0, 0, 0.4);
388- background-position-y: 30px;
389+ background: rgba(0, 0, 0, 0.0);
390+ background-position-y: 5px;
391 -webkit-transition: background-image 0.3s ease-in-out, background-color 0.3s ease-in-out, background-position 0.3s ease-in-out;
392 transition: background-image 0.3s ease-in-out, background-color 0.3s ease-in-out, background-position 0.3s ease-in-out;
393 }
394
395
396 &:checked + span {
397- background: $ubuntu_orange url(../img/tick-white@30.png) no-repeat 0.3rem 0.3rem;
398- background-size: 75% 75%;
399+ background: rgba(0,161,50, 255) url(../img/tick-white@30.png) no-repeat 0.2rem 0.2rem;
400+ background-size: 70% 70%;
401 background-position-y: 5px;
402 -webkit-transition: background-image 0.3s ease-in-out, background-color 0.3s ease-in-out, background-position 0.3s ease-in-out;
403 transition: background-image 0.3s ease-in-out, background-color 0.3s ease-in-out, background-position 0.3s ease-in-out;
404@@ -63,12 +66,12 @@
405
406 &:disabled {
407 + span {
408- background: rgba(0, 0, 0, 0.25)!important;
409+ background: rgba(0, 0, 0, 0.025)!important;
410 }
411
412 &:checked + span {
413- background: rgba(221, 72, 20, 0.5) url(../img/tick-white@30.png) no-repeat 0.3rem 0.3rem!important;
414- background-size: 75% 75%!important;
415+ background: rgba(0,161,50, 0.5) url(../img/tick-white@30.png) no-repeat 0.2em 0.2rem!important;
416+ background-size: 70% 70%!important;
417 }
418 }
419 }
420@@ -79,61 +82,61 @@
421 position: absolute;
422 left: 50%;
423 top: 50%;
424- margin: -1.1rem 0 0 -2.2rem;
425- width: 4.5rem;
426- height: 2.2rem;
427+ margin: -1.1rem 0 0 -1.2rem;
428+ width: 2.925rem;
429+ height: 1.43rem;
430 pointer-events: none;
431 overflow: hidden;
432- background: rgba(136, 136, 136, 0.3);
433+ background: rgba(136, 136, 136, 0.2);
434
435 > span.toggle-bg {
436- background: transparent url(../img/unCheckbox.png) no-repeat 0.4rem 0.35rem;
437- background-size: 90% 75%;
438+ background: transparent url(../img/unCheckbox.png) no-repeat 0.26rem 0.21rem;
439+ background-size: 90% 80%;
440 position: absolute;
441 top: 0;
442 left: 0;
443 z-index: 10;
444- width: 4.5rem;
445- height: 2.1rem;
446+ width: 2.925rem;
447+ height: 1.365rem;
448 -webkit-transition: background-image 0.3s ease-in-out;
449 transition: background-image 0.3s ease-in-out;
450 }
451
452 > span.toggle-handle {
453 @include box_sizing (border-box);
454- @include box_shadow (inset 0 1px 1px rgba(0, 0, 0, 0.3));
455+ @include box_shadow (inset 0 1px 1px rgba(0, 0, 0, 0.2));
456 position: absolute;
457- top: 2px;
458- left: 2px;
459+ top: 3px;
460+ left: 3px;
461 z-index: 2;
462- width: 2rem;
463- height: 2rem;
464+ width: 1.1rem;
465+ height: 1.1rem;
466 border-radius: 6px;
467- background: rgba(0, 0, 0, 0.4);
468+ background: rgba(0, 0, 0, 0.2);
469 -webkit-transition: background 0.3s ease-in-out, -webkit-transform 0.3s ease-in-out;
470 transition: background 0.3s ease-in-out, transform 0.3s ease-in-out;
471 }
472 }
473
474 &:checked + span.toggle {
475- background: rgba(136, 136, 136, 0.3) !important;
476+ background: rgba(136, 136, 136, 0.2) !important;
477
478 > span.toggle-bg {
479- background: transparent url(../img/Checkbox.png) no-repeat 0.4rem 0.35rem;
480- background-size: 90% 75%;
481+ background: transparent url(../img/Checkbox.png) no-repeat 0.26rem 0.2275rem;
482+ background-size: 90% 80%;
483 }
484
485 > span.toggle-handle {
486- background: rgb(221, 72, 20);
487- -webkit-transform: translate3d(38px, 0, 0);
488- transform: translate3d(38px, 0, 0);
489+ background: rgba(0,161,50, 255);
490+ -webkit-transform: translate3d(24.7px, 0, 0);
491+ transform: translate3d(24.7px, 0, 0);
492 }
493 }
494 }
495
496 label input[data-type="switch"]:disabled {
497 + span.toggle {
498- background: rgba(136, 136, 136, 0.3) !important;
499+ background: rgba(136, 136, 136, 0.03) !important;
500
501 > span.toggle-handle {
502 opacity: 0.4;
503@@ -142,7 +145,7 @@
504
505 &:checked + span.toggle > span.toggle-handle {
506 opacity: 0.4;
507- -webkit-transform: translate3d(38px, 0, 0);
508- transform: translate3d(38px, 0, 0);
509+ -webkit-transform: translate3d(24.7px, 0, 0);
510+ transform: translate3d(24.7px, 0, 0);
511 }
512-}
513\ No newline at end of file
514+}
515
516=== modified file 'examples/html5-theme/widgets/Toggles.html'
517--- examples/html5-theme/widgets/Toggles.html 2014-02-12 19:46:04 +0000
518+++ examples/html5-theme/widgets/Toggles.html 2015-12-12 11:57:23 +0000
519@@ -44,14 +44,14 @@
520
521 <div class="inset">
522 <span class="medium-font">Unchecked</span>
523- <label>
524+ <label class="checkbox">
525 <input type="checkbox">
526- <span></span>
527+ <span class="checkbox"></span>
528 </label>
529 </div>
530- <div>
531+ <div class="inset">
532 <span class="medium-font">Checked</span>
533- <label>
534+ <label class="checkbox">
535 <input type="checkbox" checked="">
536 <span></span>
537 </label>
538@@ -60,7 +60,7 @@
539 <div>
540 <span class="medium-font">Disabled</span>
541 <div class="inset" style="display:inline">
542- <label>
543+ <label class="checkbox">
544 <input type="checkbox" checked="" disabled="">
545 <span></span>
546 </label>
547@@ -131,3 +131,4 @@
548 </div>
549 </body>
550 </html>
551+

Subscribers

People subscribed via source and target branches