[openstreetmap/openstreetmap-website] Confirmation popup for Overpass API export (PR #6411)

Pablo Brasero notifications at github.com
Fri Sep 26 19:01:39 UTC 2025


@pablobm commented on this pull request.



> @@ -37,6 +37,28 @@ OSM.Export = function (map) {
     validateControls();
   }
 
+  async function showConfirmationModal() {
+    const $modal = $("#export_confirmation");
+    const $okButton = $("#ok_button");

What I mean is that, technically, an id should be unique to the whole page. So if there's another `id="download_button"` in the future, that will be against the HTML standard and perhaps malfunction with some DOM implementation. With a class, we can have as many `.download_button` as we wish in the same page.

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

Message ID: <openstreetmap/openstreetmap-website/pull/6411/review/3273535073 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20250926/b8d48d7a/attachment.htm>


More information about the rails-dev mailing list