@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");

I'm happy to use things like that, I just forget they exist 👍 

Incidentally, it's not jQuery 😉 That was a conscious choice. I think we should 
remove all jQuery.

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

Message ID: 
<openstreetmap/openstreetmap-website/pull/7132/review/[email protected]>
_______________________________________________
rails-dev mailing list
[email protected]
https://lists.openstreetmap.org/listinfo/rails-dev

Reply via email to