From 90ae0861779e01f7b6c5529d46d21bfb42d95c13 Mon Sep 17 00:00:00 2001 From: Michael M Slusarz Date: Fri, 3 Sep 2010 13:55:37 -0600 Subject: [PATCH] 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) --- imp/lib/Mime/Viewer/Html.php | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) 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']); } } -- 2.11.0