[openstreetmap/openstreetmap-website] Tags editor (PR #6758)

[Pablo Brasero]

pablobm left a comment (openstreetmap/openstreetmap-website#6758)

I agree with @tomhughes (https://github.com/openstreetmap/openstreetmap-website/pull/6758#discussion_r2744036740) that this should be moved to Rails views as much as possible. The code that puts together the content using jQuery can be implemented with a template. Not sure about [Turbo](https://turbo.hotwired.dev/) purely because I don't know what the current situation is and how complex it would be to use for sidebar content (is there any prior art?), but at the very least integrated with [the current JS router](https://github.com/openstreetmap/openstreetmap-website/blob/e73834f7756d618a4f6eee2378cedb1042addcd6/app/assets/javascripts/index.js#L292) (see for example [how the sidebar is loaded for map elements](https://github.com/openstreetmap/openstreetmap-website/blob/e73834f7756d618a4f6eee2378cedb1042addcd6/app/assets/javascripts/index/element.js#L13)).

There's also the validation, which here is happening fully on the client side. Ideally this should happen in the backend... but from what I'm seeing there's no validation applied to tags anywhere at the moment? So I don't know what the expectations are here.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/6758#issuecomment-3890634917
You are receiving this because you are subscribed to this thread.

Message ID: <openstreetmap/openstreetmap-website/pull/6758/c3890634917 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20260212/c252e5c8/attachment.htm>