@deevroman commented on this pull request.


> +    let summary = makeDetailedPrefix();
+    const mainTagsHint = makeMainTagsHint();
+    if (summary.length > 200 || changedTags.size > 1) {
+      summary = makeOnlyKeysPrefix();
+    }
+    summary = summary.replace(/; $/, "");
+    if (mainTagsHint === "") {
+      summary += ` for ${objectType}/${objectId}`;
+    } else if (removedTags.size) {
+      summary += " from" + mainTagsHint;
+    } else if (changedTags.size) {
+      summary += " of" + mainTagsHint;
+    } else if (addedTags.size) {
+      summary += " to" + mainTagsHint;
+    }

> Could we add an input box to at least give the user an option to supply a 
> more descriptive changeset comment? 

While the extra screen might be annoying for me as an advanced user, it's a 
really useful step for beginners editing. Perhaps we could even skip 
pre-filling the comment to avoid confusing non-English speakers. 

I'll experiment with this workflow


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

Message ID: 
<openstreetmap/openstreetmap-website/pull/6758/review/[email protected]>
_______________________________________________
rails-dev mailing list
[email protected]
https://lists.openstreetmap.org/listinfo/rails-dev

Reply via email to