Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Delta Between Two Patch Sets: skin/options.css

Issue 29332808: Issue 2408 - Improved accessibility of checkboxes in options page (Closed)
Left Patch Set: Rebased to 59920e6112a6 Created Dec. 17, 2015, 6:36 p.m.
Right Patch Set: Reverted styles for Advanced tab Created Jan. 25, 2016, 6:02 p.m.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
Left: Side by side diff | Download
Right: Side by side diff | Download
« no previous file with change/comment | « options.js ('k') | no next file » | no next file with change/comment »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
LEFTRIGHT
1 /* 1 /*
2 * This file is part of Adblock Plus <https://adblockplus.org/>, 2 * This file is part of Adblock Plus <https://adblockplus.org/>,
3 * Copyright (C) 2006-2015 Eyeo GmbH 3 * Copyright (C) 2006-2015 Eyeo GmbH
4 * 4 *
5 * Adblock Plus is free software: you can redistribute it and/or modify 5 * Adblock Plus is free software: you can redistribute it and/or modify
6 * it under the terms of the GNU General Public License version 3 as 6 * it under the terms of the GNU General Public License version 3 as
7 * published by the Free Software Foundation. 7 * published by the Free Software Foundation.
8 * 8 *
9 * Adblock Plus is distributed in the hope that it will be useful, 9 * Adblock Plus is distributed in the hope that it will be useful,
10 * but WITHOUT ANY WARRANTY; without even the implied warranty of 10 * but WITHOUT ANY WARRANTY; without even the implied warranty of
(...skipping 82 matching lines...) Expand 10 before | Expand all | Expand 10 after
93 } 93 }
94 94
95 button[role="checkbox"] 95 button[role="checkbox"]
96 { 96 {
97 vertical-align: top; 97 vertical-align: top;
98 width: 18px; 98 width: 18px;
99 height: 18px; 99 height: 18px;
100 margin-top: 0px; 100 margin-top: 0px;
101 -moz-margin-end: 20px; 101 -moz-margin-end: 20px;
102 -webkit-margin-end: 20px; 102 -webkit-margin-end: 20px;
103 padding: 0px 0px 0px 0px; 103 padding: 0px;
saroyanm 2016/01/19 11:19:50 Nit: Use single value. I know it's something that
Thomas Greiner 2016/01/19 15:15:05 Done.
104 border: none; 104 border: none;
105 background-color: transparent; 105 background-color: transparent;
106 background-position: -51px 0px; 106 background-position: -51px 0px;
107 } 107 }
108 108
109 button[role="checkbox"][aria-checked="true"] 109 button[role="checkbox"][aria-checked="true"]
110 { 110 {
111 background-position: -68px 0px; 111 background-position: -68px 0px;
112 } 112 }
113 113
(...skipping 167 matching lines...) Expand 10 before | Expand all | Expand 10 after
281 { 281 {
282 display: none; 282 display: none;
283 } 283 }
284 284
285 #tab-share:lang(zh) + li, 285 #tab-share:lang(zh) + li,
286 #tab-share[hidden] + li 286 #tab-share[hidden] + li
287 { 287 {
288 border-top: none; 288 border-top: none;
289 } 289 }
290 290
291 #tab-content 291 #content
292 { 292 {
293 background-color: #FFFFFF; 293 background-color: #FFFFFF;
294 border: 1px solid #CDCDCD; 294 border: 1px solid #CDCDCD;
295 border-radius: 8px; 295 border-radius: 8px;
296 box-sizing: border-box; 296 box-sizing: border-box;
297 min-width: 960px; 297 min-width: 960px;
298 padding: 0px 60px 40px 60px; 298 padding: 0px 60px 40px 60px;
299 } 299 }
300 300
301 #tab-content h1 301 #content h1
302 { 302 {
303 border-bottom: 1px solid #CDCDCD; 303 border-bottom: 1px solid #CDCDCD;
304 margin: 0px; 304 margin: 0px;
305 padding: 40px 0px 16px 0px; 305 padding: 40px 0px 16px 0px;
306 } 306 }
307 307
308 #link-version 308 #link-version
309 { 309 {
310 display: flex; 310 display: flex;
311 margin: 12px 20px; 311 margin: 12px 20px;
(...skipping 98 matching lines...) Expand 10 before | Expand all | Expand 10 after
410 { 410 {
411 background-color: #CDCDCD; 411 background-color: #CDCDCD;
412 border-radius: 5px; 412 border-radius: 5px;
413 padding: 0px 40px; 413 padding: 0px 40px;
414 } 414 }
415 415
416 .table::-webkit-scrollbar-thumb:hover, 416 .table::-webkit-scrollbar-thumb:hover,
417 #custom-wrapper::-webkit-scrollbar-thumb:hover 417 #custom-wrapper::-webkit-scrollbar-thumb:hover
418 { 418 {
419 background-color: #A1A1A1; 419 background-color: #A1A1A1;
420 }
421
422 .table input[type="checkbox"]
423 {
424 margin-top: 0px;
425 -moz-margin-end: 20px;
426 -webkit-margin-end: 20px;
427 padding: 0px 0px 0px 0px;
428 visibility: hidden;
429 }
430
431 .table input[type="checkbox"]::before
432 {
433 content: "";
434 background-position: -51px 0px;
435 height: 18px;
436 width: 18px;
437 visibility: visible;
438 }
439
440 .table input[type="checkbox"]:checked::before
441 {
442 content: "";
443 background-position: -68px 0px;
444 height: 18px;
445 padding: 0px;
446 width: 18px;
447 visibility: visible;
420 } 448 }
421 449
422 .table button.delete 450 .table button.delete
423 { 451 {
424 background-color: transparent; 452 background-color: transparent;
425 background-position: -9px -32px; 453 background-position: -9px -32px;
426 border: 0px; 454 border: 0px;
427 height: 10px; 455 height: 10px;
428 margin-top: 5px; 456 margin-top: 5px;
429 -moz-margin-end: 20px; 457 -moz-margin-end: 20px;
(...skipping 28 matching lines...) Expand all
458 } 486 }
459 487
460 .tabs.horizontal li.active 488 .tabs.horizontal li.active
461 { 489 {
462 border-bottom: 2px solid #1E8728; 490 border-bottom: 2px solid #1E8728;
463 color: black; 491 color: black;
464 font-weight: 600; 492 font-weight: 600;
465 padding-bottom: 10px; 493 padding-bottom: 10px;
466 } 494 }
467 495
468 .icon, 496 .icon, .table button[role="checkbox"], .table button.delete,
469 button[role="checkbox"], 497 #content-help a::before, #dialog-close::before,
470 .table button.delete,
471 #content-help a::before,
472 #dialog-close::before,
473 #custom-filters-add button::after, 498 #custom-filters-add button::after,
474 #dialog-body button::before 499 #dialog-body button::before
475 { 500 {
476 background-image: url(options-sprite.png); 501 background-image: url(options-sprite.png);
477 display: inline-block; 502 display: inline-block;
478 } 503 }
479 504
480 .icon-add 505 .icon-add
481 { 506 {
482 background-position: -0px -0px; 507 background-position: -0px -0px;
(...skipping 613 matching lines...) Expand 10 before | Expand all | Expand 10 after
1096 #other-language .button-add::before 1121 #other-language .button-add::before
1097 { 1122 {
1098 display: none; 1123 display: none;
1099 } 1124 }
1100 1125
1101 #other-language .display 1126 #other-language .display
1102 { 1127 {
1103 -webkit-margin-start: 10px; 1128 -webkit-margin-start: 10px;
1104 -moz-margin-start: 10px; 1129 -moz-margin-start: 10px;
1105 } 1130 }
LEFTRIGHT
« options.js ('k') | no next file » | Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Toggle Comments ('s')

Powered by Google App Engine
This is Rietveld