From: Tom Hughes <tom@compton.nu>
Date: Mon, 27 Jan 2025 19:25:50 +0000 (+0000)
Subject: Merge remote-tracking branch 'upstream/pull/5562'
X-Git-Tag: live~238
X-Git-Url: https://git.openstreetmap.org./rails.git/commitdiff_plain/8cab3c468507657cf4013aabd59d3310d7b9eea6?ds=inline;hp=-c

Merge remote-tracking branch 'upstream/pull/5562'
---

8cab3c468507657cf4013aabd59d3310d7b9eea6
diff --combined app/assets/javascripts/index/layers/data.js
index f779bbb96,8d3779043..b972adbc2
--- a/app/assets/javascripts/index/layers/data.js
+++ b/app/assets/javascripts/index/layers/data.js
@@@ -61,16 -61,11 +61,16 @@@ OSM.initializeDataLayer = function (map
            .click(add)));
    }
  
 -  function displayLoadError(message) {
 +  function displayLoadError(message, close) {
      $("#browse_status").html(
        $("<div class='p-3'>").append(
 -        $("<h2 class='flex-grow-1 text-break'>")
 -          .text(I18n.t("browse.start_rjs.load_data")),
 +        $("<div class='d-flex'>").append(
 +          $("<h2 class='flex-grow-1 text-break'>")
 +            .text(I18n.t("browse.start_rjs.load_data")),
 +          $("<div>").append(
 +            $("<button type='button' class='btn-close'>")
 +              .attr("aria-label", I18n.t("javascripts.close"))
 +              .click(close))),
          $("<div>").append(
            $("<div class='d-flex'>").append(
              $("<p class='alert alert-warning'>")
@@@ -85,10 -80,10 +85,10 @@@
  
      /*
       * Modern browsers are quite happy showing far more than 100 features in
-      * the data browser, so increase the limit to 2000 by default, but keep
+      * the data browser, so increase the limit to 4000 by default, but keep
       * it restricted to 500 for IE8 and 100 for older IEs.
       */
-     var maxFeatures = 2000;
+     var maxFeatures = 4000;
  
      /*@cc_on
        if (navigator.appVersion < 8) {
@@@ -133,16 -128,12 +133,16 @@@
          dataLoader = null;
          if (textStatus === "abort") { return; }
  
 +        function closeError() {
 +          $("#browse_status").empty();
 +        }
 +
          if (XMLHttpRequest.status === 400 && XMLHttpRequest.responseText) {
 -          displayLoadError(XMLHttpRequest.responseText);
 +          displayLoadError(XMLHttpRequest.responseText, closeError);
          } else if (XMLHttpRequest.statusText) {
 -          displayLoadError(XMLHttpRequest.statusText);
 +          displayLoadError(XMLHttpRequest.statusText, closeError);
          } else {
 -          displayLoadError(String(XMLHttpRequest.status));
 +          displayLoadError(String(XMLHttpRequest.status), closeError);
          }
        }
      });