Skip to content

Commit b813f24

Browse files
committed
Fix incorrect target handling in search comment suggestion form.
1 parent ae5e3dd commit b813f24

File tree

2 files changed

+7
-8
lines changed

2 files changed

+7
-8
lines changed

site/results.html

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ <h3 class="title">{{ r.content | join(sep=", ") }}</h3>
2828

2929
<div class="toolbar">
3030
{% if consts.enable_submissions %}
31-
<a href="#" data-from="{{ r.guid }}" class="edit" title="{{ i18n.public_suggestEdit }}"><img src="/static/edit.svg?v={{ asset_ver }}" alt="{{ i18n.public_suggestEdit }}"></a>
31+
<a href="#" data-edit-from="{{ r.guid }}" class="edit" title="{{ i18n.public_suggestEdit }}"><img src="/static/edit.svg?v={{ asset_ver }}" alt="{{ i18n.public_suggestEdit }}"></a>
3232
{% endif %}
3333
<a href="#{{ shareGuid }}" data-share-guid="{{ shareGuid }}" title="Share link"><img src="/static/share.svg?v={{ asset_ver }}" alt="Share link"></a>
3434
</div>
@@ -65,7 +65,7 @@ <h3 class="title">{{ r.content | join(sep=", ") }}</h3>
6565
</a>
6666
{% endif %}
6767
{% if consts.enable_submissions %}
68-
<a href="#" data-from="{{ r.guid }}" data-to="{{ d.guid }}" class="edit" title="{{ i18n.public_suggestEdit | default(value="Suggest edit") }}">
68+
<a href="#" data-edit-from="{{ r.guid }}" data-edit-to="{{ d.guid }}" class="edit" title="{{ i18n.public_suggestEdit | default(value="Suggest edit") }}">
6969
<img src="/static/edit.svg?v={{ asset_ver }}" alt="{{ i18n.public_suggestEdit }}">
7070
</a>
7171
{% endif %}

site/static/main.js

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -156,10 +156,9 @@
156156

157157
// Edit form.
158158
(() => {
159-
document.querySelectorAll(".edit").forEach((o) => {
160-
o.onclick = ((e) => {
159+
document.querySelectorAll("a[data-edit-from]").forEach((btn) => {
160+
btn.onclick = ((e) => {
161161
e.preventDefault();
162-
const btn = e.target;
163162

164163
// Form is already open.
165164
if (btn.close) {
@@ -168,7 +167,7 @@
168167
}
169168

170169
const form = document.querySelector(".form-comments").cloneNode(true);
171-
o.parentNode.appendChild(form);
170+
btn.parentNode.appendChild(form);
172171
form.style.display = "block";
173172

174173
const txt = form.querySelector("textarea");
@@ -192,8 +191,8 @@
192191
"Content-Type": "application/json"
193192
},
194193
body: JSON.stringify({
195-
from_guid: btn.dataset.from,
196-
to_guid: btn.dataset.to,
194+
from_guid: btn.dataset.editFrom,
195+
to_guid: btn.dataset.editTo,
197196
comments: txt.value
198197
})
199198
}).catch((err) => {

0 commit comments

Comments
 (0)