improvements on the tag admin

1 files changed, 36 insertions, 68 deletions

diff --git a/modules/tag/views/admin_tags.html.php b/modules/tag/views/admin_tags.html.php
index 18ce150e..d92f422a 100644
--- a/modules/tag/views/admin_tags.html.php
+++ b/modules/tag/views/admin_tags.html.php
@@ -1,5 +1,8 @@
 <?php defined("SYSPATH") or die("No direct script access.") ?>
 <style>
+  #gTagAdmin td {
+    border: 0;
+  }
   #gTagAdmin ul {
     padding-bottom: .3em;
   }
@@ -21,14 +24,15 @@
     padding: 0 .2em 0 .2em;
     clear: none;
     float: left;
+    margin: 0 .2em 0 0;
   }
   #gRenameTagForm input[type="submit"] {
     height: 25px;
   }
-  #gRenameTagForm a, #gEditTagForm span {
+  #gRenameTagForm a, #gRenameTagForm span {
     display: block;
     float: left;
-    padding: .2em 0 0 .3em;
+    padding: .2em .2em 0 .1em;
   }
 </style>
 <div class="gBlock">
@@ -37,79 +41,43 @@
   </h2>
 
   <? $tags_per_column = $tags->count()/5 ?>
-  <? $column_tag_number = 0 ?>
-
-  <div id="gTagAdmin" class="gBlockContent">
-    <div class="gColumn">
-      <? foreach ($tags as $i => $tag): ?>
-      <? $current_letter = strtoupper(substr($tag->name, 0, 1)) ?>
-
-      <? if ($i == 0): ?>
-      <strong><?= $current_letter ?></strong>
-      <ul>
-        <? elseif ($last_letter != $current_letter): ?>
-        <? if ($column_tag_number > $tags_per_column): ?>
-    </div>
-    <div class="gColumn">
-      <? $column_tag_number = 0 ?>
-      <? endif ?>
-    </ul>
-    <strong><?= $current_letter ?></strong>
-    <ul>
-      <? endif ?>
-
-      <li>
-        <span id="gTag-<?= $tag->id ?>" class="gEditable"
-              title="<?= t("Click to edit this tag") ?>"><?= $tag->name ?></span>
-        <span class="understate">(<?= $tag->count ?>)</span>
-        <a href="<?= url::site("admin/tags/form_delete/$tag->id") ?>" class="gDialogLink"
-          title="<?= t("Delete this tag") ?>">X</a>
-      </li>
-
-      <? $column_tag_number++ ?>
-      <? $last_letter = $current_letter ?>
-      <? endforeach ?>
-    </ul>
-
-    </div>
-  </div>
+  <? $column_tag_count = 0 ?>
 
-  <table>
+  <table id="gTagAdmin" class="gBlockContent">
+    <caption class="understate"><?= t("There are ".$tags->count()." tags") ?></caption>
     <tr>
-      <th> <?= t("Tag") ?> </th>
-      <th> <?= t("Photos") ?> </th>
-      <th> <?= t("Actions") ?> </th>
-    </tr>
-    <? foreach ($tags as $i => $tag): ?>
-    <tr class="<?= ($i % 2 == 0) ? "gEvenRow" : "gOddRow" ?>">
-      <td> <?= $tag->name ?> </td>
-      <td> <?= $tag->count ?> </td>
       <td>
-        <ul>
+        <? foreach ($tags as $i => $tag): ?>
+          <? $current_letter = strtoupper(substr($tag->name, 0, 1)) ?>
+
+          <? if ($i == 0): /* first letter */ ?>
+            <strong><?= $current_letter ?></strong>
+            <ul>
+          <? elseif ($last_letter != $current_letter): /* new letter */ ?>
+            <? if ($column_tag_count > $tags_per_column): /* new column */ ?>
+            	</td>
+              <td>
+              <? $column_tag_count = 0 ?>
+            <? endif ?>
+            
+            </ul>
+            <strong><?= $current_letter ?></strong>
+            <ul>
+          <? endif ?>
+    
           <li>
+            <span id="gTag-<?= $tag->id ?>" class="gEditable"
+                  title="<?= t("Click to edit this tag") ?>"><?= $tag->name ?></span>
+            <span class="understate">(<?= $tag->count ?>)</span>
             <a href="<?= url::site("admin/tags/form_delete/$tag->id") ?>" class="gDialogLink"
-              title="<?= t("Delete tag %tag_name", array("tag_name" => $tag->name)) ?>">
-              <?= t("delete") ?>
-            </a>
-          </li>
-          <li>
-            <a href="<?= url::site("admin/tags/form_rename/$tag->id") ?>" class="gDialogLink"
-              title="<?= t("Rename tag %tag_name", array("tag_name" => $tag->name)) ?>">
-              <?= t("rename") ?>
-            </a>
+              title="<?= t("Delete this tag") ?>">X</a>
           </li>
+    
+          <? $column_tag_count++ ?>
+          <? $last_letter = $current_letter ?>
+        <? endforeach /* $tags */ ?>
         </ul>
       </td>
     </tr>
-    <? endforeach ?>
   </table>
-
-  <div id="gTagSearch">
-    <form method="get" action="<?= url::site("admin/tags") ?>">
-      <fieldset>
-        <legend> <?= t("Search Tags") ?> </legend>
-        <input name="filter" value="<?= $filter ?>"/>
-        <input type="submit" value="<?= t("Search Tags") ?>"/>
-      </fieldset>
-    </form>
-  </div>
+</div>
\ No newline at end of file