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

Side by Side Diff: skin/desktop-options.css

Issue 29578574: Issue 5632 - Use checkboxes for toggling acceptable ads (Closed)
Patch Set: Addressed Ire's comments Created Oct. 19, 2017, 8:40 p.m.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View unified diff | Download patch
« desktop-options.js ('K') | « desktop-options.js ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
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-present eyeo GmbH 3 * Copyright (C) 2006-present 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 156 matching lines...) Expand 10 before | Expand all | Expand 10 after
167 { 167 {
168 background-image: url(icons/checkbox.png); 168 background-image: url(icons/checkbox.png);
169 display: inline-block; 169 display: inline-block;
170 } 170 }
171 171
172 button[role="checkbox"][aria-checked="true"]:not(.toggle) 172 button[role="checkbox"][aria-checked="true"]:not(.toggle)
173 { 173 {
174 background-position: 0px 18px; 174 background-position: 0px 18px;
175 } 175 }
176 176
177 button[role="checkbox"][disabled] 177 button[role="checkbox"][disabled],
178 button[role="checkbox"][aria-disabled="true"]
178 { 179 {
179 border-radius: 2px; 180 border-radius: 2px;
180 background-color: #ccc; 181 background-color: #ccc;
181 } 182 }
182 183
183 button[role="checkbox"].toggle 184 button[role="checkbox"].toggle
184 { 185 {
185 background: url(icons/toggle.svg#on); 186 background: url(icons/toggle.svg#on);
186 } 187 }
187 188
(...skipping 427 matching lines...) Expand 10 before | Expand all | Expand 10 after
615 616
616 section.cols > *:last-child 617 section.cols > *:last-child
617 { 618 {
618 flex: 3; 619 flex: 3;
619 } 620 }
620 621
621 /* 622 /*
622 Acceptable ads 623 Acceptable ads
623 */ 624 */
624 625
625 #acceptable-ads 626 #acceptable-ads ul
626 { 627 {
627 background-color: #F3F3F3; 628 position: relative;
629 padding-left: 2rem;
630 list-style: none;
628 } 631 }
629 632
630 #acceptable-ads > div 633 html[dir="rtl"] #acceptable-ads ul
631 { 634 {
632 position: relative; 635 padding-left: 0rem;
633 padding: 1.5rem; 636 padding-right: 2rem;
634 padding-left: 3.3rem;
635 padding-right: 0.9rem;
636 border: 1px solid #CDCDCD;
637 border-width: 0px 1px 1px 1px;
638 } 637 }
639 638
640 #acceptable-ads > div:first-of-type 639 #acceptable-ads button
641 { 640 {
642 border-top: 1px solid #CDCDCD; 641 position: absolute;
642 margin-top: 0.3rem;
643 left: 0rem;
643 } 644 }
644 645
645 #acceptable-ads > div > p:last-of-type 646 html[dir="rtl"] button
646 { 647 {
647 margin-bottom: 0rem; 648 left: auto;
649 right: 0rem;
648 } 650 }
649 651
650 #acceptable-ads > div input 652 #acceptable-ads label
651 {
652 position: absolute;
653 left: 0rem;
654 margin: 0.25rem 0.9rem;
655 width: 1.3rem;
656 height: 1.3rem;
657 }
658
659 #acceptable-ads > div label
660 { 653 {
661 font-weight: 700; 654 font-weight: 700;
662 font-size: 1.375rem; 655 font-size: 1.375rem;
663 } 656 }
664 657
665 #dnt 658 #dnt
666 { 659 {
667 padding: 0.8rem; 660 padding: 0.8rem;
668 border: 1px solid #099CD0; 661 border: 1px solid #099CD0;
669 } 662 }
(...skipping 783 matching lines...) Expand 10 before | Expand all | Expand 10 after
1453 1446
1454 #hide-notification::after 1447 #hide-notification::after
1455 { 1448 {
1456 background-color: #099DD1; 1449 background-color: #099DD1;
1457 } 1450 }
1458 1451
1459 #hide-notification:hover::after 1452 #hide-notification:hover::after
1460 { 1453 {
1461 background-color: #5CBCE1; 1454 background-color: #5CBCE1;
1462 } 1455 }
OLDNEW
« desktop-options.js ('K') | « desktop-options.js ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld