From 8dcdeaf23d002d2f6bd0208b73da6ca7a37cae6c Mon Sep 17 00:00:00 2001
From: Katrin Fischer <katrin.fischer.83@web.de>
Date: Tue, 19 Sep 2023 20:36:19 +0000
Subject: [PATCH] Bug 33499: (QA follow-up) Open external links in new tab

Adds target="_blank" to the vendor website and interface URLs,
as we can expect them to be external sites.

Also changes mark-up a little so that the URL is shown without
an additional space in front, lining up with the other fields.

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
---
 koha-tmpl/intranet-tmpl/prog/en/modules/acqui/supplier.tt | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/supplier.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/supplier.tt
index 2450eb2e9a..3042418cc6 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/supplier.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/supplier.tt
@@ -377,7 +377,7 @@ div.rows { padding: 1rem; }
                     <li><span class="label">Phone: </span>[% phone | html %]</li>
                     <li><span class="label">Fax: </span>[% fax | html %]</li>
                     [% IF ( url ) %]
-                        <li><span class="label">Website: </span><a href="[% url | url %]">[% url | html %]</a></li>
+                        <li><span class="label">Website: </span><a href="[% url | url %]" target="_blank">[% url | html %]</a></li>
                     [% END %]
                     [% IF ( accountnumber ) %]
                         <li><span class="label">Account number: </span>[% accountnumber | html %]</li>
@@ -425,8 +425,7 @@ div.rows { padding: 1rem; }
                         [% END %]
                         [% IF i.uri %]
                             <li>
-                                <span class="label">URI: </span>
-                                <a href="[% i.uri | url %]">[% i.uri | html %]</a>
+                                <span class="label">URI: </span><a href="[% i.uri | url %]" target="_blank">[% i.uri | html %]</a>
                             </li>
                         [% END %]
                         [% IF i.login %]
-- 
2.39.5