@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/[email protected]>
_______________________________________________
rails-dev mailing list
[email protected]
https://lists.openstreetmap.org/listinfo/rails-dev

Reply via email to