Rework API reference pages

chrome/common/extensions/docs/static/sass/_api.scss

+.api {
+
+  color: #333;
+  font-size: 14px;
+
+  .apisummary {

[not at google - send to devlin, 2014/01/03 23:30:24]

nit: style I usually see in chromium is for class names to be separated with a
-, like api-summary, api-reference, inner-table, etc.

[Renato Mangini (chromium), 2014/02/06 03:19:17]

Done.

+    td, th {

[not at google - send to devlin, 2014/01/03 23:30:24]

nit: likewise chromium style usually to separate rules on different lines,
though when they're really simple rules it probably doesn't matter so much.

[Renato Mangini (chromium), 2014/02/06 03:19:17]

Done.

+      padding: 5px 10px;
+    }
+    a {
+      color: inherit;
+      font-weight: normal;
+    }
+    td:hover a, td:hover a * {
+      cursor: pointer;
+      color: $link-color;
+      font-weight: bold;
+    }
+  }
+  .api_reference {
+    table.innerTable {
+      margin: 10px 0;
+
+      td, th {
+        padding: 5px 10px;
+        border: 1px solid #eee;
+      }
+      th {
+        background: none;
+      }
+    }
+
+    td, th {
+      border: 1px solid #eee;
+    }
+    th {
+      background: #fafafa;
+    }
+
+    h2 {
+      background-color: darken($gray-light, 5%);
+      padding: $default-padding;
+      margin-left: $default-padding * -1;
+      margin-right: $default-padding * -1;
+    }
+
+    h3 {
+      margin-top: rhythm(2);
+    }
+
+    .description {
+      margin-left: 20px;
+    }
+  }
+
+

[not at google - send to devlin, 2014/01/03 23:30:24]

nit: can the use of blank lines in this file be consistent? I don't really mind
what it is, but it could be
- no blank lines here
- blank line between every {..} block unless they're related (e.g. td, th {...}
th {...})

[Renato Mangini (chromium), 2014/02/06 03:19:17]

Done.

+}