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>
This commit is contained in:
parent
f6b9f25a4f
commit
8dcdeaf23d
1 changed files with 2 additions and 3 deletions
|
@ -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 %]
|
||||
|
|
Loading…
Reference in a new issue