[openstreetmap/openstreetmap-website] Fix issue #6614: Support spaces in @mentions using quotes (PR #6616)
Anshul Zilpe
notifications at github.com
Sat Dec 20 17:39:23 UTC 2025
aNsHuL5217 left a comment (openstreetmap/openstreetmap-website#6616)
@pnorman I gave the `@"foo'` edge case a shot, but getting it to work safely (without breaking URLs or HTML) made the regex extremely complex and unreadable.
I'm thinking of sticking with the simpler implementation. In my opinion, the trade-off of having hard-to-maintain code isn't worth it for such a rare edge case. The current version handles all standard and smart quotes correctly.
I've reverted to the cleaner version—let me know if you're okay with this approach!
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/6616#issuecomment-3677984616
You are receiving this because you are subscribed to this thread.
Message ID: <openstreetmap/openstreetmap-website/pull/6616/c3677984616 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20251220/1c5e1fe9/attachment.htm>
More information about the rails-dev
mailing list