<div style="display: flex; flex-wrap: wrap; white-space: pre-wrap; align-items: center; "><img height="20" width="20" style="border-radius:50%; margin-right: 4px;" decoding="async" src="https://avatars.githubusercontent.com/u/36066?s=20&v=4" /><strong>pablobm</strong> left a comment <a href="https://github.com/openstreetmap/openstreetmap-website/issues/6407#issuecomment-3371681894">(openstreetmap/openstreetmap-website#6407)</a></div>
<p dir="auto">Regarding that example link to booking.com, that's partially a different problem:</p>
<ul dir="auto">
<li>It's a different problem in that the user entered a URL with semicolons, which technically they shouldn't have (but I can't blame them).</li>
<li>It's the same problem in that the incorrectly-splitted strings shouldn't be rendered as links.</li>
</ul>
<p dir="auto">My impression is that multiple values (semicolon-separated) should be rendered differently. Perhaps in different lines, hopefully not all in a single line as that's not visually obvious. Then each separate link could get whichever treatment we see appropriate: not render as a link, show a warning sign, etc.</p>
<p dir="auto">Perhaps editors like iD should offer to enter multiple values in separate form fields, then join them with semicolons behind the scenes. This way the editor could automatically escape the semicolons to <code class="notranslate">%3B</code>, avoiding this problem. Having said that, in this case the changeset was produced with Level0, so that wouldn't have helped.</p>
<p dir="auto">Looked into why <code class="notranslate">website=</code> and <code class="notranslate">source=</code> are treated differently. At a technical level, the website code delegates some decisions here to <a href="https://github.com/JOSM/tag2link">https://github.com/JOSM/tag2link</a>. This in turn has:</p>
<ul dir="auto">
<li>A special case for <code class="notranslate">website</code> (<a href="https://github.com/JOSM/tag2link/blob/e09148b173b7ef6c74988a28e4ef1a218e4cb374/index.json#L3003-L3006">code</a>), telling it to render the value exactly as given. Therefore <code class="notranslate">www.example.com</code> remains unchanged, without being automatically prefixed with a scheme.</li>
<li>No mention of <code class="notranslate">source</code>. The default behaviour here is to apply a smart linkification using <a href="https://github.com/vmg/rinku">https://github.com/vmg/rinku</a>. (Code rabbit hole starts at <a href="https://github.com/openstreetmap/openstreetmap-website/blob/02f458e7e8092b2ebff556d6538bd4a45861de08/app/helpers/browse_tags_helper.rb#L51">browse_tags_helper.rb</a>)<br>
As for whether there's a reason for this, I don't know. I'm guessing it's a historic accident, and both should be made to render the same way.</li>
</ul>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />Reply to this email directly, <a href="https://github.com/openstreetmap/openstreetmap-website/issues/6407#issuecomment-3371681894">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AAK2OLMFRIXBXBMR7GF6HDL3WJVQVAVCNFSM6AAAAACHDI33ZGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZTGNZRGY4DCOBZGQ">unsubscribe</a>.<br />You are receiving this because you are subscribed to this thread.<img src="https://github.com/notifications/beacon/AAK2OLO3G6PP4ELI23QR5XL3WJVQVA5CNFSM6AAAAACHDI33ZGWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTWI67EGM.gif" height="1" width="1" alt="" /><span style="color: transparent; font-size: 0; display: none; visibility: hidden; overflow: hidden; opacity: 0; width: 0; height: 0; max-width: 0; max-height: 0; mso-hide: all">Message ID: <span><openstreetmap/openstreetmap-website/issues/6407/3371681894</span><span>@</span><span>github</span><span>.</span><span>com></span></span></p>

<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/openstreetmap/openstreetmap-website/issues/6407#issuecomment-3371681894",
"url": "https://github.com/openstreetmap/openstreetmap-website/issues/6407#issuecomment-3371681894",
"name": "View Issue"
},
"description": "View this Issue on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>