From 6e68533fec22941562d16f6651199767bb148fa7 Mon Sep 17 00:00:00 2001
From: Jan Schneider <jan@horde.org>
Date: Fri, 16 Oct 2009 14:19:25 +0200
Subject: [PATCH] Fix popup link.

---
 nag/templates/list/task_headers.inc | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/nag/templates/list/task_headers.inc b/nag/templates/list/task_headers.inc
index cfbf96f64..99dbf44b1 100644
--- a/nag/templates/list/task_headers.inc
+++ b/nag/templates/list/task_headers.inc
@@ -63,10 +63,11 @@ function doPrefsUpdate(column, sortDown)
   <th id="s<?php echo Nag::SORT_CATEGORY ?>"<?php if ($sortby == Nag::SORT_CATEGORY) echo ' class="' . $sortdirclass . '"' ?> width="10%">
    <?php echo Horde::widget(Horde::applicationUrl(Horde_Util::addParameter($baseurl, 'sortby', Nag::SORT_CATEGORY)), _("Sort by Category"), 'sortlink', '', '', _("Cat_egory")) ?>&nbsp;
    <?php
-   if (Horde_Auth::getAuth() && (!$GLOBALS['prefs']->isLocked('categories') ||
-                           !$GLOBALS['prefs']->isLocked('category_colors'))) {
+   if (Horde_Auth::getAuth() &&
+       (!$GLOBALS['prefs']->isLocked('categories') ||
+        !$GLOBALS['prefs']->isLocked('category_colors'))) {
        $categoryUrl = Horde_Util::addParameter(Horde::getServiceLink('options', 'horde'), array('group' => 'categories'));
-       echo ' ' . Horde::link($categoryUrl, _("Edit categories and colors"), '', '_blank', Horde::popupJs($categoryUrl, array('urlencode' => true)) . 'return false;') . Horde::img('colorpicker.png', _("Edit categories and colors"), '', $GLOBALS['registry']->getImageDir('horde')) . '</a>';
+       echo ' ' . Horde::link($categoryUrl, _("Edit categories and colors"), '', '_blank', 'Horde.popup({url:this.href}); return false;') . Horde::img('colorpicker.png', _("Edit categories and colors"), '', $GLOBALS['registry']->getImageDir('horde')) . '</a>';
    }
    ?>
   </th>
-- 
2.11.0