[openstreetmap/openstreetmap-website] Adds note tags support (PR #5344)
Nenad Vujicic
notifications at github.com
Mon Dec 9 15:50:07 UTC 2024
@nenad-vujicic commented on this pull request.
> + note.tags.each do |k, v|
+ xml.tag(:k => k, :v => v)
+ end
Thanks for catching this. Honestly, I'm not sure about what could be the best solution here. I believe we have multiple options:
1. Adding Atom + XHTML namespaces and displaying tags like in [user's history feed](https://www.openstreetmap.org/user/Nenad%20Vujicic/history/feed) (I'm a little bit afraid to add new namespaces because of client support)
2. Using e.g. <dc:subject> for simplified, flattened representation of tags (the easiest and the most portable, but not so nice visual representation of tags)
3. Creating custom or some other namespace or some other solution?
Can you suggest what could be the best approach here?
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/5344#discussion_r1876232635
You are receiving this because you are subscribed to this thread.
Message ID: <openstreetmap/openstreetmap-website/pull/5344/review/2489231608 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20241209/8edadf72/attachment.htm>
More information about the rails-dev
mailing list