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

Tom Hughes notifications at github.com
Mon Jun 8 18:15:39 UTC 2026


@tomhughes commented on this pull request.

The first commit can go away, given the second commit undoes it.

> @@ -24,6 +24,12 @@
     <div class="mb-3">
       <%= text_area_tag "text", "", :class => "form-control", :size => "40x10", :maxlength => "2000", :placeholder => t(".advice") %>
     </div>
+    <div class="new-note-error alert alert-danger d-none">
+      <details>
+        <summary><%= t(".error") %></summary>
+        <p class="new-note-error__detail"></p>

Is there some reason for the slightly weird formatting of the class name here? The mix of hyphens and underscores, and the double underscore?

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

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


More information about the rails-dev mailing list