Wrap all admin views in g-block and g-block content. This provides the means to visually separate the view's title and description from everything else. Primary admin view title should always be h1, and only one h1 per view. Removed some unused admin CSS id's.

1 files changed, 42 insertions, 41 deletions

diff --git a/modules/tag/views/admin_tags.html.php b/modules/tag/views/admin_tags.html.php
index 7771f7fe..67ba4d95 100644
--- a/modules/tag/views/admin_tags.html.php
+++ b/modules/tag/views/admin_tags.html.php
@@ -1,5 +1,5 @@
 <?php defined("SYSPATH") or die("No direct script access.") ?>
-<script>
+<script type="text/javascript">
   var TAG_RENAME_URL = <?= html::js_string(url::site("admin/tags/rename/__ID__")) ?>;
   $("document").ready(function() {
     // using JS for adding link titles to avoid running t() for each tag
@@ -15,49 +15,50 @@
   var cancel_i18n = <?= html::js_string(t("cancel")->for_html_attr()) ?>;
 </script>
 
-<h2>
-  <?= t("Tag Admin") ?>
-</h2>
-
 <? $tags_per_column = $tags->count()/5 ?>
 <? $column_tag_count = 0 ?>
 
-<table id="g-tag-admin">
-  <caption class="g-understate">
-    <?= t2("There is one tag", "There are %count tags", $tags->count()) ?>
-  </caption>
-  <tr>
-    <td>
-      <? foreach ($tags as $i => $tag): ?>
-        <? $current_letter = strtoupper(mb_substr($tag->name, 0, 1)) ?>
-
-        <? if ($i == 0): /* first letter */ ?>
-          <strong><?= html::clean($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 ?>
+<div class="g-block">
+  <h1> <?= t("Tag Admin") ?> </h1>
+  
+  <div class="g-block-content">
+    <table id="g-tag-admin">
+      <caption class="g-understate">
+        <?= t2("There is one tag", "There are %count tags", $tags->count()) ?>
+      </caption>
+      <tr>
+        <td>
+          <? foreach ($tags as $i => $tag): ?>
+            <? $current_letter = strtoupper(mb_substr($tag->name, 0, 1)) ?>
 
-          </ul>
-          <strong><?= html::clean($current_letter) ?></strong>
-          <ul>
-        <? endif ?>
+            <? if ($i == 0): /* first letter */ ?>
+              <strong><?= html::clean($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><?= html::clean($current_letter) ?></strong>
+              <ul>
+            <? endif ?>
 
-        <li>
-          <span id="g-tag-<?= $tag->id ?>" class="g-editable g-tag-name"><?= html::clean($tag->name) ?></span>
-          <span class="g-understate">(<?= $tag->count ?>)</span>
-          <a href="<?= url::site("admin/tags/form_delete/$tag->id") ?>"
-             class="g-dialog-link g-delete-link g-button">
-              <span class="ui-icon ui-icon-trash"><?= t("Delete this tag") ?></span></a>
-        </li>
+                <li>
+                  <span class="g-editable g-tag-name" rel="<?= $tag->id ?>"><?= html::clean($tag->name) ?></span>
+                  <span class="g-understate">(<?= $tag->count ?>)</span>
+                  <a href="<?= url::site("admin/tags/form_delete/$tag->id") ?>"
+                      class="g-dialog-link g-delete-link g-button">
+                    <span class="ui-icon ui-icon-trash"><?= t("Delete this tag") ?></span></a>
+                </li>
 
-        <? $column_tag_count++ ?>
-        <? $last_letter = $current_letter ?>
-      <? endforeach /* $tags */ ?>
-      </ul>
-    </td>
-  </tr>
-</table>
+            <? $column_tag_count++ ?>
+            <? $last_letter = $current_letter ?>
+          <? endforeach /* $tags */ ?>
+          </ul>
+        </td>
+      </tr>
+    </table>
+  </div>
+</div>