css hack n' slash, continued: New styles for buttons and selects.

chrome/browser/resources/print_preview/print_preview.css

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 4168a445138d486e89b73063b7e61878a816091b..bf1af78479e917801fa65595aacab8db66cd3092 100644
--- a/chrome/browser/resources/print_preview/print_preview.css
+++ b/chrome/browser/resources/print_preview/print_preview.css
@@ -32,7 +32,7 @@ header {
 }
 
 #print-header {
-  -webkit-padding-end: 16px;
+  -webkit-padding-end: 14px;
   padding-bottom: 10px;
   padding-top: 10px;
 }
@@ -125,11 +125,11 @@ button.loading {
   cursor: progress;
 }
 
-button.default {
+#print-preview button.default {
   font-weight: bold;
 }
 
-button.default:not(:focus):not(:disabled) {
+#print-preview button.default:not(:focus):not(:disabled) {
   border-color: #808080;
 }
 
@@ -246,7 +246,7 @@ label {
 
 #copies {
   position: relative;
-  width: 1.5em;
+  width: 2em;

[dpapad, 2012/03/02 16:34:10]

Could we make this 2.75em, so that it can fit the maximum allowed number of
digits (which is 3)? 

[Dan Beam, 2012/03/02 17:58:43]

2.75em - why not 3em? confident that no number digit will ever take up as much
space as an m in any font? (even ... monospace! dun dun dun!)

[Evan Stade, 2012/03/02 19:46:20]

no, actually, in CJK fonts most characters are wider than an em. However they
still use arabic numerals AFAIK, and I don't think it's possible to change this
font to monospace (well anything is possible but I don't think it's easy). I
think 2.75em is a safe bet, and if you keep going wider it eventually just looks
bad; if 2.75 is not enough the consequences are non-disastrous.

[Dan Beam, 2012/03/02 20:24:42]

it will be possible when users are allowed to change the font of all WebUI
pages, but I agree we should probably optimize for the common case (of 2.75em
being the best looking).

 }
 
 #copies.invalid {
@@ -261,7 +261,7 @@ label {
   font-weight: 600;
   min-width: 0;
   position: relative;
-  width: 25px;
+  width: 2em;
 }
 
 #increment:focus,
@@ -411,7 +411,8 @@ html:not([toolkit=views]) .button-strip {
   -webkit-box-direction: reverse;
 }
 
-.button-strip button {
+#print-preview .button-strip button {

[Dan Beam, 2012/03/02 20:24:42]

if we're depending on not conflicting with other .button-strip styles why is
this necessary? specificity? there should be nothing you need to override, eh?

[Evan Stade, 2012/03/02 21:47:02]

yep, specificity

+  -webkit-margin-start: 4px;
   display: block;
 }