[openstreetmap/openstreetmap-website] Display error message if note creation fails (PR #7132)

Pablo Brasero notifications at github.com
Tue Jun 9 10:08:05 UTC 2026


@pablobm commented on this pull request.



> @@ -141,20 +151,33 @@ OSM.NewNote = function (map) {
       const location = newNoteMarker.getLatLng().wrap();
       const text = content.find("textarea").val();
 
+      errorPanel = document.querySelector(".new-note-error");
+      errorPanel.classList.add("d-none");

@hlfan - On second thought: you are right that this project makes a conscious decision to use BS for styles. As I understand it, we should use as little custom CSS as possible, so let's stick to this JS-driven style.

@tomhughes - Good point. Changed it to jQuery style. I'll create a separate issue to discuss jQuery.

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

Message ID: <openstreetmap/openstreetmap-website/pull/7132/review/4457567536 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20260609/ba11c39e/attachment-0001.htm>


More information about the rails-dev mailing list