@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/2489231...@github.com>
_______________________________________________
rails-dev mailing list
rails-dev@openstreetmap.org
https://lists.openstreetmap.org/listinfo/rails-dev
