[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:
https://github.com/openstreetmap/openstreetmap-website/issues/2069#issuecomment-441094568
-------------- 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