[openstreetmap/openstreetmap-website] Added error handling for object display on map (PR #6353)

[Anton Khorev]

@AntonKhorev commented on this pull request.



> +    function displayLoadError(message, close) {
+      $("#browse_status").html(
+        $("<div class='p-3'>").append(
+          $("<div class='d-flex'>").append(
+            $("<h2 class='flex-grow-1 text-break'>")
+              .text(OSM.i18n.t("browse.start_rjs.load_data")),
+            $("<div>").append(
+              $("<button type='button' class='btn-close'>")
+                .attr("aria-label", OSM.i18n.t("javascripts.close"))
+                .click(close))),
+          $("<p class='alert alert-warning'>")
+            .text(OSM.i18n.t("browse.start_rjs.feature_error", { message: message }))));
     }

Why don't you first introduce and use `OSM.displayLoadError` instead of adding this function and then deleting it?

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

Message ID: <openstreetmap/openstreetmap-website/pull/6353/review/3155533563 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20250826/8d51c115/attachment.htm>