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

Unified Diff: skin/options.css

Issue 29323156: Issue 2410 - Improved accessibility of dialogs in options page (Closed)
Patch Set: Created July 31, 2015, 3:02 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 | « 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
@@ -467,7 +467,8 @@
.icon, .table input[type="checkbox"]::before, .table button.delete,
#content-help a::before, #dialog-close::before,
-#custom-filters-add button::after
+#custom-filters-add button::after,
+#dialog-body button::before
{
background-image: url(options-sprite.png);
display: inline-block;
@@ -497,8 +498,8 @@
width: 18px;
}
-.icon-arrow,
-#content-help a::before
+#content-help a::before,
+#dialog-body button::before
{
background-position: 0px -42px;
content: "";
@@ -506,6 +507,8 @@
height: 11px;
vertical-align: middle;
width: 7px;
+ -moz-margin-end: 12px;
+ -webkit-margin-end: 12px;
}
.controls
@@ -988,9 +991,11 @@
margin: 4px 24px;
}
-#dialog h3
+#dialog h3,
+#dialog label
{
font-size: 14px;
+ font-weight: 600;
margin: 0px;
}
@@ -1033,22 +1038,18 @@
padding: 12px 0px;
}
-#dialog .button-wrapper
+#dialog-body button
{
background-color: #F5F5F5;
+ border: none;
+ color: #3A7BA6;
+ cursor: pointer;
+ display: block;
+ font-family: inherit;
margin-top: 8px;
padding: 10px 16px;
- cursor: pointer;
- width: auto;
-}
-
-#dialog .button-wrapper span:not(.icon)
-{
- color: #3A7BA6;
- display: inline-block;
- -moz-margin-start: 15px;
- -webkit-margin-start: 15px;
- vertical-align: top;
+ text-align: left;
+ width: 100%;
}
#dialog .url
@@ -1071,6 +1072,10 @@
#other-language .button-add
{
+ background-color: transparent;
+ margin: 0px;
+ padding: 0px;
+ width: auto;
-webkit-border-end: 1px solid #CDCDCD;
-moz-border-end: 1px solid #CDCDCD;
-webkit-padding-end: 10px;
@@ -1079,6 +1084,11 @@
-moz-padding-start: 0px;
}
+#other-language .button-add::before
+{
+ display: none;
+}
+
#other-language .display
{
-webkit-margin-start: 10px;
« no previous file with comments | « options.js ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld