Standardize size of WebUI controls for touch and non-touch.

chrome/browser/resources/shared/css/widgets.css

Index: chrome/browser/resources/shared/css/widgets.css
diff --git a/chrome/browser/resources/shared/css/widgets.css b/chrome/browser/resources/shared/css/widgets.css
index 5214921cce397ec4fe84552d7610da1e4eb865cf..907d27047c265e817d186dc6c1cd8024e463f61f 100644
--- a/chrome/browser/resources/shared/css/widgets.css
+++ b/chrome/browser/resources/shared/css/widgets.css
@@ -65,6 +65,14 @@ select {
   background-repeat: no-repeat;
 }
 
+:-webkit-any(button,
+             input[type='button'],
+             input[type='submit'],
+             select):not(.custom-appearance):not(.link-button) {
+  margin-bottom: 6px;
+  margin-top: 6px;

[Evan Stade, 2012/08/10 02:44:16]

how much verification did you do that this doesn't make anything look crazy?

[kevers, 2012/08/10 13:56:03]

This rule has been in effect for a while now with the touch optimized UI. 
Without it, a vertical stack of buttons are nearly touching, which creates some
tap errors.  The alternative is to use larger buttons; however, UX feedback
suggests keeping the button sizes smaller if possible.

One place where this pattern is used is on the keyboard shortcuts dialog. 
Attached a screenshot in the email.  Can do some experimentation to determine
how small the margin can be before tap error rates climb.

[Evan Stade, 2012/08/10 21:21:08]

sorry, I wasn't clear enough. I didn't mean that the margin would be too big and
look weird. I was thinking about all the many pages that use buttons and these
other elements, and wondering if one of the many pages would be broken by this
change.

The fact that it has been on for a while with the touch optimized build doesn't
give me much confidence, although it's better than nothing. But I'm still wary
without a manual pass over all webui pages because 1) we have a ton of pages and
2) poor spacing or padding or alignment in the past has often persisted for
several releases in a row, showing that most people don't care to file a bug
report when they notice issues, or simply don't consciously notice the issues.

so basically I was looking for an answer like "I manually inspected every page
in chrome://chrome, every dialog I could find in settings, downloads, feedback,
etc."

[kevers, 2012/08/13 17:18:46]

Removed rule in favor of standardizing the formatting of settings pages instead.
 Added "settings-row" as appropriate to fix the vertical spacing.

+}
+
 html[dir='rtl'] select {
   background-position: center left;
 }
@@ -291,14 +299,11 @@ input:disabled:-webkit-any([type='password'],
  *   </div>
  */
 
-.checkbox,
-.radio {
-  margin: 0.65em 0;
-}
-
 :-webkit-any(.checkbox, .radio) label {
   /* Don't expand horizontally: <http://crbug.com/112091>. */
   display: -webkit-inline-box;
+  padding-bottom: 7px;
+  padding-top: 7px;
 }
 
 :-webkit-any(.checkbox, .radio) label input ~ span {