[openstreetmap/openstreetmap-website] Bootstrap modal instead of alert() (PR #6263)
Marwin Hochfelsner
notifications at github.com
Sun Jul 27 19:55:08 UTC 2025
@hlfan commented on this pull request.
> @@ -10,14 +10,21 @@
URL.revokeObjectURL(url);
};
+ OSM.showAlert = function (message) {
+ const modalBody = document.getElementById("osm_alert_message");
+ modalBody.textContent = message;
+ const alertModal = new bootstrap.Modal(document.getElementById("osm_alert_modal"));
+ alertModal.show();
+ };
+
class DownloadUtil {
static async handleExportSuccess(fetchResponse, filename) {
try {
const blob = await fetchResponse.response.blob();
OSM.downloadBlob(blob, filename);
} catch (err) {
// eslint-disable-next-line no-alert
These lines are no longer needed then
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/6263#pullrequestreview-3059670860
You are receiving this because you are subscribed to this thread.
Message ID: <openstreetmap/openstreetmap-website/pull/6263/review/3059670860 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20250727/06016c36/attachment.htm>
More information about the rails-dev
mailing list