Reorder "action links" on chrome://extensions

chrome/browser/resources/extensions/extensions.html

Index: chrome/browser/resources/extensions/extensions.html
diff --git a/chrome/browser/resources/extensions/extensions.html b/chrome/browser/resources/extensions/extensions.html
index 56dd88ec8fc7fb9d6443a5f29bdc3df0614c9511..65c970c8b81cb78bce37ed5c2cbc47f4413398b8 100644
--- a/chrome/browser/resources/extensions/extensions.html
+++ b/chrome/browser/resources/extensions/extensions.html
@@ -133,14 +133,21 @@
           <span class="extension-title"></span>
           <span class="extension-version"></span>
         </div>
-        <p class="extension-description">
-          <span></span>
+        <p class="extension-description"></p>
+        <div class="action-links">
+          <a class="options-link" i18n-content="extensionSettingsOptions"
+              href="#" hidden></a>

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

why is this an <a> if the [href] isn't useful?  why not .link-button?

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

For that most terrible of reasons: I copied the code.  Fixed them to make it
better. :)

Note, however, this results in the good/bad effect of only having the link
underline for the details/developer website links.  This is
Good: It keeps in line with what we do everywhere else, saying "underline if it
means navigate away", otherwise don't."
Bad:  It looks kinda weird to have one "randomly" underlined link in the row.

Until told otherwise (possibly by UI team), I'm going to assume the good
outweighs the bad in this case.

           <a class="permissions-link"
               i18n-content="extensionSettingsPermissions"
               href="#">
           </a>
           <a class="site-link" target="_parent" hidden></a>
-        </p>
+          <a class="launch-link" i18n-content="extensionSettingsLaunch"
+              href="#" hidden></a>
+          <a class="reload-link"
+              i18n-content="extensionSettingsReloadUnpacked"
+              href="#" hidden></a>
+        </div>
         <div class="permanent-warnings">
           <div class="extension-warnings" hidden>
             <span i18n-content="extensionSettingsWarningsTitle"></span>
@@ -212,19 +219,12 @@
               <span i18n-content="extensionSettingsAllowOnAllUrls"></span>
             </label>
           </div>
-          <span class="optional-controls-disableable">
-            <label class="file-access-control" hidden>
+          <div class="checkbox file-access-control" hidden>
+            <label>
               <input type="checkbox">
               <span i18n-content="extensionSettingsAllowFileAccess"></span>
             </label>
-            <a class="reload-link"
-                i18n-content="extensionSettingsReloadUnpacked"
-                href="#" hidden></a>
-            <a class="launch-link" i18n-content="extensionSettingsLaunch"
-                href="#" hidden></a>
-          </span>
-          <a class="options-link" i18n-content="extensionSettingsOptions"
-              href="#" hidden></a>
+          </div>
         </div>
         <div class="butter-bar"
             i18n-values=".innerHTML:extensionSettingsIncognitoWarning" hidden>