[openstreetmap/openstreetmap-website] Properly mark `tel:` link on multiple phone numbers in the POI (#2069)

Tom Hughes notifications at github.com
Thu Nov 22 17:50:54 UTC 2018

A simple statement of the problem would probably have sufficed ;-)

For anybody interested in looking at this the relevant code is https://github.com/openstreetmap/openstreetmap-website/blob/master/app/helpers/browse_helper.rb#L181 but it's not a completely trivial change because it is both recognising it as a phone number and also determining the link target as a single step and that will need to be split up so that we can detect a semicolon delimited list and then loop doing the substitutions.

You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20181122/371b9aca/attachment.html>

More information about the rails-dev mailing list