Tweak upgrader html and css so that we set opacity properly for rows

that are done, and disable the upgrade link when there're no actions
to take.

3 files changed, 10 insertions, 4 deletions

diff --git a/modules/gallery/controllers/upgrader.php b/modules/gallery/controllers/upgrader.php
index a86e8af9..48769bce 100644
--- a/modules/gallery/controllers/upgrader.php
+++ b/modules/gallery/controllers/upgrader.php
@@ -43,7 +43,7 @@ class Upgrader_Controller extends Controller {
     $view->can_upgrade = identity::active_user()->admin || $session->get("can_upgrade");
     $view->upgrade_token = $upgrade_token;
     $view->available = module::available();
-    $view->done = ($available_upgrades == 0);
+    $view->done = $available_upgrades == 0;
     print $view;
   }
 
diff --git a/modules/gallery/css/upgrader.css b/modules/gallery/css/upgrader.css
index 97d00941..01689a97 100644
--- a/modules/gallery/css/upgrader.css
+++ b/modules/gallery/css/upgrader.css
@@ -37,7 +37,7 @@ td {
 }
 
 tr.current td {
-  color: #999;
+  opacity: 0.5;
   font-style: italic;
 }
 
@@ -87,7 +87,7 @@ div.button a {
   text-decoration: none;
 }
 
-div.button:hover {
+div.button-active:hover {
   background: #ccc;
 }
 
diff --git a/modules/gallery/views/upgrader.html.php b/modules/gallery/views/upgrader.html.php
index 44234ec0..90e1c5b6 100644
--- a/modules/gallery/views/upgrader.html.php
+++ b/modules/gallery/views/upgrader.html.php
@@ -58,11 +58,17 @@
           <? endforeach ?>
         </table>
 
-        <div class="button">
+        <? if ($done): ?>
+        <div class="button muted">
+          <?= t("Upgrade all") ?>
+        </div>
+        <? else: ?>
+        <div class="button button-active">
           <a href="<?= url::site("upgrader/upgrade") ?>">
             <?= t("Upgrade all") ?>
           </a>
         </div>
+        <? endif ?>
 
         <? if (@$inactive): ?>
         <p class="<?= $done ? "muted" : "" ?>">