@hlfan commented on this pull request.


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

I thought we could prefill most on the server side so it looks more like this:

<details>
  <summary>There was an error when creating the note</summary>
  <p>403: Forbidden</p>
</details>

```html
<details>
  <summary>There was an error when creating the note</summary>
  <p>403: Forbidden</p>
</details>
```

So `details>summary` would be set in the template and `details>p` with err in 
JS.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/7132#discussion_r3363718014
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