<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/6631117?s=20&v=4" /><strong>Ltrlg</strong> left a comment <a href="https://github.com/openstreetmap/openstreetmap-website/issues/6611#issuecomment-3641484052">(openstreetmap/openstreetmap-website#6611)</a></div>
<p dir="auto">Sorry, I meant “changeset discussion comment”, forgot that the term is used for both.</p>
<p dir="auto"><a class="user-mention notranslate" data-hovercard-type="user" data-hovercard-url="/users/tomhughes/hovercard" data-octo-click="hovercard-link-click" data-octo-dimensions="link_type:self" href="https://github.com/tomhughes">@tomhughes</a> Convention is not syntax. Markdown is a (somewhat) formalized syntax that was first inspired by preexisting conventions for text-only writing and then influenced conventions in return. Granted, it might not be the most common convention, but I did use it in comments before and retroactively changing rendering to make it look bad seems wrong.</p>
<p dir="auto"><a class="user-mention notranslate" data-hovercard-type="user" data-hovercard-url="/users/hlfan/hovercard" data-octo-click="hovercard-link-click" data-octo-dimensions="link_type:self" href="https://github.com/hlfan">@hlfan</a> Thanks! The set of characters used in common OSM keys is pretty limited¹, maybe that part should use a positive set instead of a negative one?</p>
<p dir="auto">¹ Arguably even the values are limited, since this is linking to wiki pages and values with more unusual characters are probably not much documented. The proper thing to do imho would be to use the same logic as in tags tables to choose linking to full tag / key only / not at all, but this would be more work and your quick fix is a good step forward.</p>
<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/6611#issuecomment-3641484052">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AAK2OLMPYTPB5OM3SACY4ET4BFIQZAVCNFSM6AAAAACOWY4HJWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZTMNBRGQ4DIMBVGI">unsubscribe</a>.<br />You are receiving this because you are subscribed to this thread.<img src="https://github.com/notifications/beacon/AAK2OLPEYM4GEAII5FSXYZD4BFIQZA5CNFSM6AAAAACOWY4HJWWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTWZBSRRI.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/6611/3641484052</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/6611#issuecomment-3641484052",
"url": "https://github.com/openstreetmap/openstreetmap-website/issues/6611#issuecomment-3641484052",
"name": "View Issue"
},
"description": "View this Issue on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>