From: Tom Hughes Date: Tue, 28 Jan 2025 18:23:33 +0000 (+0000) Subject: Merge remote-tracking branch 'upstream/pull/5566' X-Git-Tag: live~235 X-Git-Url: https://git.openstreetmap.org./rails.git/commitdiff_plain/eaa0540f2298782df81dd5d005128394c8e06633?ds=sidebyside;hp=-c Merge remote-tracking branch 'upstream/pull/5566' --- eaa0540f2298782df81dd5d005128394c8e06633 diff --combined app/assets/javascripts/index/layers/data.js index 812c35be9,5da57d4af..c10c6ed21 --- a/app/assets/javascripts/index/layers/data.js +++ b/app/assets/javascripts/index/layers/data.js @@@ -44,7 -44,7 +44,7 @@@ OSM.initializeDataLayer = function (map } } - function displayFeatureWarning(count, limit, add, cancel) { + function displayFeatureWarning(num_features, add, cancel) { $("#browse_status").html( $("
").append( $("
").append( @@@ -55,7 -55,7 +55,7 @@@ .attr("aria-label", I18n.t("javascripts.close")) .click(cancel))), $("

") - .text(I18n.t("browse.start_rjs.feature_warning", { num_features: count, max_features: limit })), + .text(I18n.t("browse.start_rjs.feature_warning", { num_features })), $("") .val(I18n.t("browse.start_rjs.load_data")) .click(add))); @@@ -85,9 -85,18 +85,9 @@@ /* * Modern browsers are quite happy showing far more than 100 features in - * the data browser, so increase the limit to 4000 by default, but keep - * it restricted to 500 for IE8 and 100 for older IEs. + * the data browser, so increase the limit to 4000 by default. */ - var maxFeatures = 4000; - - /*@cc_on - if (navigator.appVersion < 8) { - maxFeatures = 100; - } else if (navigator.appVersion < 9) { - maxFeatures = 500; - } - @*/ + const maxFeatures = 4000; if (dataLoader) dataLoader.abort(); @@@ -111,7 -120,7 +111,7 @@@ if (features.length < maxFeatures) { addFeatures(); } else { - displayFeatureWarning(features.length, maxFeatures, addFeatures, cancelAddFeatures); + displayFeatureWarning(features.length, addFeatures, cancelAddFeatures); } if (map._objectLayer) {