<p>Thanks for the feedback.  After looking at the usage of the / in existing data, I think it was premature for me to disallow it from links.</p>
<p>It looks like the documented standard (<a href="https://www.itu.int/rec/T-REC-E.123-200102-I/en" rel="nofollow">1</a>, <a href="http://publications.europa.eu/code/en/en-390300.htm" rel="nofollow">2</a>, <a href="https://tools.ietf.org/html/rfc3966#section-3" rel="nofollow">3</a>) is to use the / to separate alternative phone numbers without repeating the beginning part that is the same.  In this case, the existing code which passes the / directly into the tel: URL would be incorrect.  However, in the existing data, the / is widely used in a few countries (Germany, Austria, Hungary, Romania) to separate the region code from the local number.  In this case, what the existing code does may be correct.  It would be better to get feedback from someone in one of these countries before changing this.</p>
<p>So I think I'd better change this to keep the existing treatment of the /.  I'll submit a new pull request to hopefully make things cleaner.</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/2071#issuecomment-441469965">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/ABWnLTA6DKq6V4XNi2RtbbMoMvJiBZgEks5uyvxagaJpZM4Yxjxu">mute the thread</a>.<img src="https://github.com/notifications/beacon/ABWnLfPwBUjJQwPUWPn6HhWnVcYC0X2kks5uyvxagaJpZM4Yxjxu.gif" height="1" width="1" alt="" /></p>
<script type="application/json" data-scope="inboxmarkup">{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/openstreetmap/openstreetmap-website","title":"openstreetmap/openstreetmap-website","subtitle":"GitHub repository","main_image_url":"https://assets-cdn.github.com/images/email/message_cards/header.png","avatar_image_url":"https://assets-cdn.github.com/images/email/message_cards/avatar.png","action":{"name":"Open in GitHub","url":"https://github.com/openstreetmap/openstreetmap-website"}},"updates":{"snippets":[{"icon":"PERSON","message":"@krubokrubo in #2071: Thanks for the feedback.  After looking at the usage of the / in existing data, I think it was premature for me to disallow it from links.\r\n\r\nIt looks like the documented standard ([1](https://www.itu.int/rec/T-REC-E.123-200102-I/en), [2](http://publications.europa.eu/code/en/en-390300.htm), [3](https://tools.ietf.org/html/rfc3966#section-3)) is to use the / to separate alternative phone numbers without repeating the beginning part that is the same.  In this case, the existing code which passes the / directly into the tel: URL would be incorrect.  However, in the existing data, the / is widely used in a few countries (Germany, Austria, Hungary, Romania) to separate the region code from the local number.  In this case, what the existing code does may be correct.  It would be better to get feedback from someone in one of these countries before changing this.\r\n\r\nSo I think I'd better change this to keep the existing treatment of the /.  I'll submit a new pull request to hopefully make things cleaner."}],"action":{"name":"View Pull Request","url":"https://github.com/openstreetmap/openstreetmap-website/pull/2071#issuecomment-441469965"}}}</script>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/openstreetmap/openstreetmap-website/pull/2071#issuecomment-441469965",
"url": "https://github.com/openstreetmap/openstreetmap-website/pull/2071#issuecomment-441469965",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
},
{
"@type": "MessageCard",
"@context": "http://schema.org/extensions",
"hideOriginalBody": "false",
"originator": "AF6C5A86-E920-430C-9C59-A73278B5EFEB",
"title": "Re: [openstreetmap/openstreetmap-website] Show tel: links on multiple phone numbers separated by ; (#2071)",
"sections": [
{
"text": "",
"activityTitle": "**krubokrubo**",
"activityImage": "https://assets-cdn.github.com/images/email/message_cards/avatar.png",
"activitySubtitle": "@krubokrubo",
"facts": [

]
}
],
"potentialAction": [
{
"name": "Add a comment",
"@type": "ActionCard",
"inputs": [
{
"isMultiLine": true,
"@type": "TextInput",
"id": "IssueComment",
"isRequired": false
}
],
"actions": [
{
"name": "Comment",
"@type": "HttpPOST",
"target": "https://api.github.com",
"body": "{\n\"commandName\": \"IssueComment\",\n\"repositoryFullName\": \"openstreetmap/openstreetmap-website\",\n\"issueId\": 2071,\n\"IssueComment\": \"{{IssueComment.value}}\"\n}"
}
]
},
{
"targets": [
{
"os": "default",
"uri": "https://github.com/openstreetmap/openstreetmap-website/pull/2071#issuecomment-441469965"
}
],
"@type": "OpenUri",
"name": "View on GitHub"
},
{
"name": "Unsubscribe",
"@type": "HttpPOST",
"target": "https://api.github.com",
"body": "{\n\"commandName\": \"MuteNotification\",\n\"threadId\": 415644782\n}"
}
],
"themeColor": "26292E"
}
]</script>