Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(4350)

Unified Diff: chrome/browser/resources/print_preview/print_preview.css

Issue 8537034: print preview: fix focus outline for increment/decrement/copies (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: sync Created 9 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/resources/print_preview/print_preview.css
diff --git a/chrome/browser/resources/print_preview/print_preview.css b/chrome/browser/resources/print_preview/print_preview.css
index 46d2002e1c9f88e24662516d4c1bfb93209f7d3f..ab179e8332b1a06ae2b3ce00b603e56529116578 100644
--- a/chrome/browser/resources/print_preview/print_preview.css
+++ b/chrome/browser/resources/print_preview/print_preview.css
@@ -132,36 +132,6 @@ button.default:not(:focus):not(:disabled) {
border-color: #808080;
}
-#increment,
-#decrement {
- -webkit-padding-end: 8px;
- -webkit-padding-start: 8px;
- font-weight: 600;
- min-width: 0;
- position: relative;
- width: 25px;
-}
-
-#increment {
- -webkit-margin-start: -11px !important;
- border-radius: 0;
-}
-
-#decrement {
- -webkit-margin-start: -5px;
- border-bottom-left-radius: 0;
- border-bottom-right-radius: 3px;
- border-top-left-radius: 0;
- border-top-right-radius: 3px;
-}
-
-html[dir='rtl'] #decrement {
- border-bottom-left-radius: 3px;
- border-bottom-right-radius: 0;
- border-top-left-radius: 3px;
- border-top-right-radius: 0;
-}
-
span.hint {
-webkit-transition: color 200ms;
background: white;
@@ -293,7 +263,8 @@ label {
}
#copies {
- width: 2em;
+ position: relative;
+ width: 1.5em;
}
#copies.invalid {
@@ -301,6 +272,42 @@ label {
color: #8c1414;
}
+#increment,
+#decrement {
+ -webkit-padding-end: 8px;
+ -webkit-padding-start: 8px;
+ font-weight: 600;
+ min-width: 0;
+ position: relative;
+ width: 25px;
+}
+
+#increment:focus,
+#decrement:focus,
+#copies:focus {
+ z-index: 1;
+}
+
+#increment {
+ -webkit-margin-start: -5px;
+ border-radius: 0;
+}
+
+#decrement {
+ -webkit-margin-start: -5px;
+ border-bottom-left-radius: 0;
+ border-bottom-right-radius: 3px;
+ border-top-left-radius: 0;
+ border-top-right-radius: 3px;
+}
+
+html[dir='rtl'] #decrement {
+ border-bottom-left-radius: 3px;
+ border-bottom-right-radius: 0;
+ border-top-left-radius: 3px;
+ border-top-right-radius: 0;
+}
+
/* PDF view */
#print-preview #mainview {
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698