From 3be0846ae20cbd5bcfb52d2b17306e621bd5170e Mon Sep 17 00:00:00 2001
From: =?utf8?q?Paul=20H=C3=A4nsch?= <paul@plutz.net>
Date: Wed, 11 Oct 2023 15:55:14 +0200
Subject: [PATCH] list locations in move form

---
 handlers/60_move_rename_delete.sh | 11 +++++++++--
 l10n/de.sh                        |  4 ++--
 themes/default.css                |  6 ++++++
 3 files changed, 17 insertions(+), 4 deletions(-)

diff --git a/handlers/60_move_rename_delete.sh b/handlers/60_move_rename_delete.sh
index 49651a2..eefd9b4 100755
--- a/handlers/60_move_rename_delete.sh
+++ b/handlers/60_move_rename_delete.sh
@@ -52,7 +52,7 @@ l10n_movepage >/dev/null 2>&1 \
   cat <<-EOF
 	  <h1>Move Page</h1>
 	  <p class="pageid">$(HTML "${page}")</p>
-	  <input name="newlocation" value="$(HTML "$location")" placeholder="New Location">
+	  <input list="locations" autocomplete="off" name="newlocation" value="$(HTML "$location")" placeholder="New Location">
 	  <ul>
 	    <li>A page with the same name must not already exist at the new location.</li>
 	    <li>You must have permission to create new pages at this location.</li>
@@ -68,7 +68,7 @@ l10n_renamepage >/dev/null 2>&1 \
   cat <<-EOF
 	  <h1>Rename Page</h1>
 	  <p class="pageid">$(HTML "${page}")</p>
-	  <input name="newname" value="$(HTML "$name")" placeholder="New Name">
+	  <input autocomplete="off" name="newname" value="$(HTML "$name")" placeholder="New Name">
 	  <ul>
 	    <li>A page with the new name must not already exist.</li>
 	    <li>You must have permission to create new pages at this location.</li>
@@ -125,6 +125,13 @@ else case "${PATH_INFO}" in
       theme_page - <<-EOF
 	<form method=POST id=movepage>
 	  <input type=hidden name=session_id value="$SESSION_KEY">
+	  <datalist id="locations">
+	  $(page_glob / -1 |while read loc; do
+	    [ "$loc" = "$page" ] && continue
+	    acl_write "$loc" || continue
+	    printf '\n    <option>%s</option>' "$(HTML "$loc")"
+	  done)
+	  </datalist>
 	  $(l10n_movepage)
 	</form>
 	EOF
diff --git a/l10n/de.sh b/l10n/de.sh
index 40f2292..8842a90 100644
--- a/l10n/de.sh
+++ b/l10n/de.sh
@@ -227,7 +227,7 @@ l10n_movepage(){  # TRANSLATION
   cat <<-EOF
 	  <h1>Seite verschieben</h1>
 	  <p class="pageid">$(HTML "${page}")</p>
-	  <input name="newlocation" value="$(HTML "$location")" placeholder="Neuer Ort">
+	  <input list="locations" autocomplete="off" name="newlocation" value="$(HTML "$location")" placeholder="Neuer Ort">
 	  <ul>
 	    <li>Eine Seite mit dem selben Namen darf nicht schon vorhanden sein.</li>
 	    <li>Sie müssen die Berechtigung haben, Seiten an diesem Ort anzulegen.</li>
@@ -242,7 +242,7 @@ l10n_renamepage(){  # TRANSLATION
   cat <<-EOF
 	  <h1>Seite Umbenennen</h1>
 	  <p class="pageid">$(HTML "${page}")</p>
-	  <input name="newname" value="$(HTML "$name")" placeholder="Neuer Name">
+	  <input autocomplete="off" name="newname" value="$(HTML "$name")" placeholder="Neuer Name">
 	  <ul>
 	    <li>Eine Seite mit dem neuen Namen darf nicht schon vorhanden sein.</li>
 	    <li>Sie müssen die Berechtigung haben, Seiten an diesem Ort anzulegen.</li>
diff --git a/themes/default.css b/themes/default.css
index 4508104..ae1cdf9 100644
--- a/themes/default.css
+++ b/themes/default.css
@@ -107,6 +107,12 @@ main > form#deletepage input#delsub:checked + label + ul li.delsub {
   text-decoration: line-through;
 }
 
+main > form#movepage input,
+main > form#renamepage input {
+  min-width: 30em;
+  max-width: 100%;
+}
+
 [id$="/[revision]"] main .revisions,
 [id$="/[attachment]"] main > .attachment.list {
   margin: 1em;
-- 
2.39.2