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

Unified Diff: skin/desktop-options.css

Issue 29578574: Issue 5632 - Use checkboxes for toggling acceptable ads (Closed)
Patch Set: Fixed the duplication Created Oct. 20, 2017, 6:56 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « desktop-options.js ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: skin/desktop-options.css
===================================================================
--- a/skin/desktop-options.css
+++ b/skin/desktop-options.css
@@ -174,7 +174,8 @@
background-position: 0px 18px;
}
-button[role="checkbox"][disabled]
+button[role="checkbox"][disabled],
+button[role="checkbox"][aria-disabled="true"]
{
border-radius: 2px;
background-color: #ccc;
@@ -622,41 +623,33 @@
Acceptable ads
*/
-#acceptable-ads
+#acceptable-ads ul
{
- background-color: #F3F3F3;
+ position: relative;
+ padding-left: 2rem;
+ list-style: none;
}
-#acceptable-ads > div
+html[dir="rtl"] #acceptable-ads ul
{
- position: relative;
- padding: 1.5rem;
- padding-left: 3.3rem;
- padding-right: 0.9rem;
- border: 1px solid #CDCDCD;
- border-width: 0px 1px 1px 1px;
+ padding-left: 0rem;
+ padding-right: 2rem;
}
-#acceptable-ads > div:first-of-type
+#acceptable-ads button
{
- border-top: 1px solid #CDCDCD;
+ position: absolute;
+ margin-top: 0.3rem;
+ left: 0rem;
}
-#acceptable-ads > div > p:last-of-type
+html[dir="rtl"] button
{
- margin-bottom: 0rem;
+ left: auto;
+ right: 0rem;
}
-#acceptable-ads > div input
-{
- position: absolute;
- left: 0rem;
- margin: 0.25rem 0.9rem;
- width: 1.3rem;
- height: 1.3rem;
-}
-
-#acceptable-ads > div label
+#acceptable-ads label
{
font-weight: 700;
font-size: 1.375rem;
« no previous file with comments | « desktop-options.js ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld