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

<hr>

<p>In <a href="https://github.com/openstreetmap/openstreetmap-website/pull/4361#discussion_r1401888690">app/assets/stylesheets/common.scss</a>:</p>
<pre style='color:#555'>> @@ -50,6 +50,10 @@ small, aside {
 time[title] {
   text-decoration: underline dotted;
 }
+time[title]:active::after,
+time[title]:focus::after {
+  content: " (" attr(title) ")";
</pre>
<p dir="auto">This isn't particularly i18n friendly, because it hard-codes the characters used for parenthesis and adds a space character, which might not be wanted in some languages e.g. japanese etc (I don't know for sure, but that's why we leave this to our translators, who are the experts!).</p>
<p dir="auto">If you want to show some text that's not a perfect match for the existing title attribute, then it would be better to add the new text as a different attribute, and use the i18n system to construct that new display text.</p>
<p dir="auto">I'm also concerned about whether this works properly for RTL text, but I haven't assessed that.</p>
<p dir="auto">Overall, this feels like a "neat hack", but using CSS interactions to inject content into the text doesn't feel to me like a particularly robust method.</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/pull/4361#pullrequestreview-1744137182">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AAK2OLMCFR4OKQFWIZOLAMTYFXM3LAVCNFSM6AAAAAA7V5HPWKVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTONBUGEZTOMJYGI">unsubscribe</a>.<br />You are receiving this because you are subscribed to this thread.<img src="https://github.com/notifications/beacon/AAK2OLNGB4MN7OBOR6DL7LTYFXM3LA5CNFSM6AAAAAA7V5HPWKWGG33NNVSW45C7OR4XAZNRKB2WY3CSMVYXKZLTORJGK5TJMV32UY3PNVWWK3TUL5UWJTTH6VV54.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/pull/4361/review/1744137182</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/pull/4361#pullrequestreview-1744137182",
"url": "https://github.com/openstreetmap/openstreetmap-website/pull/4361#pullrequestreview-1744137182",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>