@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