Fix a bug where we were using <? ?> tags in css to figure out whether we

should gray out an element.

2 files changed, 7 insertions, 7 deletions

diff --git a/modules/gallery/css/upgrader.css b/modules/gallery/css/upgrader.css
index 64e4ca07..97d00941 100644
--- a/modules/gallery/css/upgrader.css
+++ b/modules/gallery/css/upgrader.css
@@ -112,8 +112,8 @@ div#confirmation div {
   background: #eee;
 }
 
-.gray_on_done {
-  opacity: <?= $done ? "0.5" : "1" ?>;
+.muted {
+  opacity: 0.5;
 }
 
 pre {
diff --git a/modules/gallery/views/upgrader.html.php b/modules/gallery/views/upgrader.html.php
index 80acc2a4..44234ec0 100644
--- a/modules/gallery/views/upgrader.html.php
+++ b/modules/gallery/views/upgrader.html.php
@@ -29,11 +29,11 @@
           });
         </script>
         <? endif ?>
-        <p class="gray_on_done">
+        <p class="<?= $done ? "muted" : "" ?>">
           <?= t("Welcome to the Gallery upgrader.  One click and you're done!") ?>
         </p>
         <table>
-          <tr class="gray_on_done">
+          <tr class="<?= $done ? "muted" : "" ?>">
             <th> <?= t("Module name") ?> </th>
             <th> <?= t("Installed version") ?> </th>
             <th> <?= t("Available version") ?> </th>
@@ -58,17 +58,17 @@
           <? endforeach ?>
         </table>
 
-        <div class="button gray_on_done">
+        <div class="button">
           <a href="<?= url::site("upgrader/upgrade") ?>">
             <?= t("Upgrade all") ?>
           </a>
         </div>
 
         <? if (@$inactive): ?>
-        <p class="gray_on_done">
+        <p class="<?= $done ? "muted" : "" ?>">
           <?= t("The following modules are inactive and don't require an upgrade.") ?>
         </p>
-        <ul class="gray_on_done">
+        <ul class="<?= $done ? "muted" : "" ?>">
           <? foreach ($available as $module): ?>
           <? if (!$module->active): ?>
           <li>