From: Michael M Slusarz <slusarz@curecanti.org>
Date: Fri, 3 Sep 2010 19:55:37 +0000 (-0600)
Subject: Don't remember why we don't want targets with hash fragments to open in new target... 
X-Git-Url: https://git.internetallee.de/?a=commitdiff_plain;h=90ae0861779e01f7b6c5529d46d21bfb42d95c13;p=horde.git

Don't remember why we don't want targets with hash fragments to open in new target, but it doesn't make sense (and breaks things)
---

diff --git a/imp/lib/Mime/Viewer/Html.php b/imp/lib/Mime/Viewer/Html.php
index 9c6ccb150..71c9ffeba 100644
--- a/imp/lib/Mime/Viewer/Html.php
+++ b/imp/lib/Mime/Viewer/Html.php
@@ -301,16 +301,14 @@ class IMP_Mime_Viewer_Html extends Horde_Mime_Viewer_Html
             case 'a':
             case 'area':
                 /* Convert links to open in new windows. Ignore
-                 * mailto: links, links that have an "#xyz" anchor,
-                 * and links that already have a target. */
+                 * mailto: links and links that already have a target. */
                 if ($node->hasAttribute('href')) {
                     $url = parse_url($node->getAttribute('href'));
                     if (isset($url['scheme']) && ($url['scheme'] == 'mailto')) {
                         /* We don't include Horde.popup() in IFRAME, so need
                          * to use 'simple' links. */
                         $node->setAttribute('href', IMP::composeLink($node->getAttribute('href'), array(), true));
-                    } elseif (!$node->hasAttribute('target') &&
-                              empty($url['fragment'])) {
+                    } elseif (!$node->hasAttribute('target')) {
                         $node->setAttribute('target', $this->_imptmp['target']);
                     }
                 }