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

Unified Diff: skin/options.css

Issue 29332808: Issue 2408 - Improved accessibility of checkboxes in options page (Closed)
Patch Set: Created Jan. 19, 2016, 3:10 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
« options.js ('K') | « options.js ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: skin/options.css
===================================================================
--- a/skin/options.css
+++ b/skin/options.css
@@ -92,6 +92,25 @@
box-sizing: border-box;
}
+button[role="checkbox"]
+{
+ vertical-align: top;
+ width: 18px;
+ height: 18px;
+ margin-top: 0px;
+ -moz-margin-end: 20px;
+ -webkit-margin-end: 20px;
+ padding: 0px;
+ border: none;
+ background-color: transparent;
+ background-position: -51px 0px;
+}
+
+button[role="checkbox"][aria-checked="true"]
+{
+ background-position: -68px 0px;
+}
+
.option-name
{
display: flex;
@@ -269,7 +288,7 @@
border-top: none;
}
-#tab-content
+#content
{
background-color: #FFFFFF;
border: 1px solid #CDCDCD;
@@ -279,7 +298,7 @@
padding: 0px 60px 40px 60px;
}
-#tab-content h1
+#content h1
{
border-bottom: 1px solid #CDCDCD;
margin: 0px;
@@ -400,34 +419,6 @@
background-color: #A1A1A1;
}
-.table input[type="checkbox"]
saroyanm 2016/01/25 14:45:44 I think this style changes also belong to blocking
Thomas Greiner 2016/01/25 18:04:21 Done.
-{
- margin-top: 0px;
- -moz-margin-end: 20px;
- -webkit-margin-end: 20px;
- padding: 0px 0px 0px 0px;
- visibility: hidden;
-}
-
-.table input[type="checkbox"]::before
-{
- content: "";
- background-position: -51px 0px;
- height: 18px;
- width: 18px;
- visibility: visible;
-}
-
-.table input[type="checkbox"]:checked::before
-{
- content: "";
- background-position: -68px 0px;
- height: 18px;
- padding: 0px;
- width: 18px;
- visibility: visible;
-}
-
.table button.delete
{
background-color: transparent;
@@ -474,8 +465,11 @@
padding-bottom: 10px;
}
-.icon, .table input[type="checkbox"]::before, .table button.delete,
-#content-help a::before, #dialog-close::before,
+.icon,
+button[role="checkbox"],
+.table button.delete,
+#content-help a::before,
+#dialog-close::before,
#custom-filters-add button::after,
#dialog-body button::before
{
« options.js ('K') | « options.js ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld