<p></p>
<p><b>@hiddewie</b> commented on this pull request.</p>

<hr>

<p>In <a href="https://github.com/openstreetmap/openstreetmap-website/pull/2889#discussion_r511596780">app/views/oauth_clients/index.html.erb</a>:</p>
<pre style='color:#555'>> @@ -15,7 +15,7 @@
   </thead>
   <% @tokens.each do |token| %>
     <tr>
-      <td><%= link_to token.client_application.name, token.client_application.url %></td>
+      <td><%= link_to token.client_application.name, token.client_application.url, target: "_blank" %></td>
</pre>
<p>I would argue that any link that does not remain in the current application should be opened in a new tab. This makes users less likely to lose their way if they click on the OAuth application name. There are many more external links which should maybe also be opened in a new tab. It would be even better to annotate them with an icon like below to notify users of such links.</p>
<p><a target="_blank" rel="noopener noreferrer" href="https://user-images.githubusercontent.com/1073881/97108386-8cb96880-16cd-11eb-9e3d-5b3ea7c940c3.png"><img src="https://user-images.githubusercontent.com/1073881/97108386-8cb96880-16cd-11eb-9e3d-5b3ea7c940c3.png" alt="image" style="max-width:100%;"></a></p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/openstreetmap/openstreetmap-website/pull/2889#discussion_r511596780">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AAK2OLNM7GSAW27OVBFFZJLSMQRFNANCNFSM4SUL4G6Q">unsubscribe</a>.<img src="https://github.com/notifications/beacon/AAK2OLIB73CGEQO4DJSJGDLSMQRFNA5CNFSM4SUL4G62YY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOD3DQEWY.gif" height="1" width="1" alt="" /></p>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/openstreetmap/openstreetmap-website/pull/2889#discussion_r511596780",
"url": "https://github.com/openstreetmap/openstreetmap-website/pull/2889#discussion_r511596780",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>