Reorder "action links" on chrome://extensions

chrome/browser/resources/extensions/extensions.css

Index: chrome/browser/resources/extensions/extensions.css
diff --git a/chrome/browser/resources/extensions/extensions.css b/chrome/browser/resources/extensions/extensions.css
index 8f170047d0fa456d86a9a591d4da7ca807950eca..60515b5473ee4a6332a7448b8a290ee01c10c358 100644
--- a/chrome/browser/resources/extensions/extensions.css
+++ b/chrome/browser/resources/extensions/extensions.css
@@ -187,6 +187,11 @@ html[dir=rtl] #footer-section {
   min-height: 48px;
 }
 
+.extension-list-item a {
+  -webkit-margin-start: 0.5em;
+  display: inline-block;
+}
+
 html[dir='rtl'] .extension-list-item {
   background-position: right;
 }
@@ -215,12 +220,23 @@ html[dir='rtl'] .extension-list-item {
   white-space: normal;
 }
 
+.action-links {
+  -webkit-margin-after: 0.5em;
+}
+
+.action-links > a {

[Dan Beam, 2014/09/04 23:01:19]

why do you need > ?

[Dan Beam, 2014/09/04 23:01:19]

if not having needless margin on the end matter, you probably want:

  .action-links a:not(:last-of-type) {
    -webkit-margin-end: 1em;
  }

if they're all siblings.

[Devlin, 2014/09/04 23:28:09]

Couple of things:
- It shouldn't matter, so I'd be inclined not to put it in (it's kind of a
mouthful).
- not(:last-of-type) doesn't really work, because the last of the type (the
reload link) isn't always the last *visible* of type, so there could still be
trailing margins.  I guess if we wanted to get really fancy, we could instead
target the first visible child using the semi-hack of "any child that's visible
and has no visible parent", but that seems like overkill.

[Devlin, 2014/09/04 23:28:09]

Don't need.  Just like to be specific, and right now, all the a's we care about
are immediate children.  I'll take it out, though.

Update: Slight change to accommodate link buttons, lemme know if it should be
tweaked again.

+  -webkit-margin-end: 1em;
+  -webkit-margin-start: 0;
+}
+
 .extension-details {
   -webkit-box-flex: 1;
+  -webkit-padding-end: 7px;
   -webkit-padding-start: 55px;
 }
 
 .extension-description,
+.extension-version,
 .extension-list-item-wrapper.inactive-extension .extension-details,
 .location-text,
 .blacklist-text,
@@ -263,11 +279,6 @@ html[dir='rtl'] .extension-list-item {
   padding-top: 7px;
 }
 
-.extension-list-item a {
-  -webkit-margin-start: 0.5em;
-  display: inline-block;
-}
-
 .install-warnings a {
   -webkit-margin-start: 0;
 }
@@ -376,10 +387,6 @@ html[dir='rtl'] #extension-settings .trash {
   padding: 5px 0 5px 5px;
 }
 
-.may-not-modify .optional-controls .optional-controls-disableable {
-  display: none;
-}
-
 /* Supervised users */
 
 .page:not(.profile-is-supervised) .profile-is-supervised-banner,