From 71de3cfc34412fb9ef376e586729ce9443f5ff8f Mon Sep 17 00:00:00 2001
From: "Michael J. Rubinsky" <mrubinsk@horde.org>
Date: Fri, 30 Jan 2009 11:51:29 -0500
Subject: [PATCH] Tweak output of ajax tag actions - don't show section header
 if no tags etc...

---
 kronolith/lib/Imple/TagActions.php | 5 +++--
 kronolith/templates/panel.inc      | 4 ++--
 2 files changed, 5 insertions(+), 4 deletions(-)

diff --git a/kronolith/lib/Imple/TagActions.php b/kronolith/lib/Imple/TagActions.php
index 89a5923ac..1c7015130 100644
--- a/kronolith/lib/Imple/TagActions.php
+++ b/kronolith/lib/Imple/TagActions.php
@@ -89,7 +89,6 @@ class Kronolith_Imple_TagActions extends Kronolith_Imple
     private function _getTagHtml($tagger, $id, $type)
     {
         $tags = $tagger->getTags($id, 'calendar');
-        $html = '';
         $js = '';
 
         if ($type == 'calendar') {
@@ -103,11 +102,13 @@ class Kronolith_Imple_TagActions extends Kronolith_Imple
             $hasEdit = $event->hasPermission(PERMS_EDIT, Auth::getAuth());
         }
 
+        $html = count($tags) ? '<h5>' . _("Tags") . '</h5>': '';
+        $html .= '<ul>';
         foreach ($tags as $tag_id => $tag) {
             $html .= '<li>' .  $tag . ($hasEdit ? '<a href="#" onclick="removeTag(\'' . $id . '\', \'' . $type . '\',' . $tag_id . ', \'' . Horde::url('imple.php', true) . '\'); Event.stop(event)" id="remove' . md5($id . $tag_id) . '">' . Horde::img('delete-small.png', _("Remove Tag"), '', $GLOBALS['registry']->getImageDir('horde')) . '</a>' : '') . '</li>';
         }
 
-        return $html;
+        return $html . '</ul><div class="clear">&nbsp;</div>';
     }
 
 }
diff --git a/kronolith/templates/panel.inc b/kronolith/templates/panel.inc
index 537063f22..7a0e1cd53 100644
--- a/kronolith/templates/panel.inc
+++ b/kronolith/templates/panel.inc
@@ -44,10 +44,10 @@ $tagger = new Kronolith_Tagger();
  <ul id="mycalendars">
 <?php foreach ($my_calendars as $id => $cal): ?>
   <li><label><input type="checkbox" class="checkbox" name="display_cal[]" value="<?php echo htmlspecialchars($id) ?>"<?php echo (in_array($id, $display_calendars) ? ' checked="checked"' : '') . ' /> ' . htmlspecialchars($cal->get('name')) ?></label> <?php echo $info_img ?>
-   <div class="panel-tags">
+   <div class="panel-tags" id="tags-<?php echo $id?>">
    <?php $tags = $tagger->getTags($id, 'calendar'); ?>
    <h5><?php echo (count($tags) ? _("Tags") : '');?></h5>
-   <ul id="tags-<?php echo $id?>" style="margin-left: 10px;">
+   <ul style="margin-left: 10px;">
     <?php foreach ($tags as $tag_id => $tag): ?>
     <li><?php echo $tag . ($cal->hasPermission(Auth::getAuth(), PERMS_EDIT) ? '<a href="#" id="remove' . md5($id . $tag_id) . '">' . Horde::img('delete-small.png', _("Remove Tag"), '', $registry->getImageDir('horde')) . '</a>' : '') . '</li>'?>
     <?php
-- 
2.11.0