From 64aac361cdd8b9c5a9b8c44d8a975f5c68660ea9 Mon Sep 17 00:00:00 2001
From: Asger Geel Weirsoee <asger@weircon.dk>
Date: Tue, 20 Oct 2026 21:12:21 +0200
Subject: [PATCH] =?UTF-8?q?V=C3=A6rtsside:=20hvert=20spil=20er=20sin=20ege?=
 =?UTF-8?q?n=20startknap?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Fejl: spillet blev valgt med radioknapper, hvor Fup og Fakta var afkrydset
som standard. En TV-fjernbetjening flytter fokus uden at afkrydse, og
Enter/OK på en radio sender formularen med den afkrydsede radio, ikke den
fokuserede. Så "Tegn og Gæt" + OK gav game=fof og et Fup og Fakta-rum.

Nu er hvert spil en submit-knap med name="game", så den knap, man trykker
på, er det spil, der startes. Testen fejler på den gamle skabelon.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
---
 party/static/party/party.css        |  1 +
 party/templates/party/host_new.html | 13 ++++++-------
 party/tests/test_tegn.py            |  9 +++++++++
 3 files changed, 16 insertions(+), 7 deletions(-)

diff --git a/party/static/party/party.css b/party/static/party/party.css
index c6d1142..1e1fa8d 100644
--- a/party/static/party/party.css
+++ b/party/static/party/party.css
@@ -900,6 +900,7 @@ button, input { font-family: inherit; }
 
 /* ---------- nyt spil: vælg spørgsmålspakke ---------- */
 .new-game { display: flex; flex-direction: column; align-items: center; gap: 20px; }
+.new-game .games { display: flex; gap: 16px; flex-wrap: wrap; justify-content: center; }
 .packs { display: flex; gap: 8px; margin: 0; padding: 0; border: 0; flex-wrap: wrap; justify-content: center; }
 .packs legend { width: 100%; margin-bottom: 8px; color: var(--muted); font-weight: 600; }
 .packs label { cursor: pointer; }
diff --git a/party/templates/party/host_new.html b/party/templates/party/host_new.html
index d402cc7..a528660 100644
--- a/party/templates/party/host_new.html
+++ b/party/templates/party/host_new.html
@@ -5,13 +5,8 @@
   <h1>Weirsøe Party Protocol</h1>
   <div class="sub"><span class="b-fup">Fup</span> <span class="b-og">og</span> <span class="b-fakta">Fakta</span></div>
   <form method="post" action="/host/new" class="new-game">{% csrf_token %}
-    <fieldset class="packs games">
-      <legend>Spil</legend>
-      <label><input type="radio" name="game" value="fof" checked><span>Fup og Fakta</span></label>
-      <label><input type="radio" name="game" value="tegn"><span>Tegn og Gæt ✏️</span></label>
-    </fieldset>
     <fieldset class="packs">
-      <legend>Spørgsmål</legend>
+      <legend>Spørgsmål til Fup og Fakta</legend>
       <label><input type="radio" name="pack" value="alle" checked><span>Alle</span></label>
       <label><input type="radio" name="pack" value="familie"><span>Familie</span></label>
       <label><input type="radio" name="pack" value="nørd"><span>Nørd</span></label>
@@ -23,7 +18,11 @@
         <p class="pack-hint">Slang, memes og dansk pop med zoomer-speaker. Narret = tag en tår, finalen er en shot.</p>
       </details>
     </fieldset>
-    <button type="submit" class="btn btn-go">Start et nyt spil</button>
+    {# hvert spil er sin egen knap: Enter/OK på en fokuseret radio sender formularen med den afkrydsede, ikke den fokuserede #}
+    <div class="games">
+      <button type="submit" name="game" value="fof" class="btn btn-go">Start Fup og Fakta</button>
+      <button type="submit" name="game" value="tegn" class="btn btn-go">Start Tegn og Gæt ✏️</button>
+    </div>
   </form>
   {% if username %}<p class="who">Logget ind som {{ username }}</p>{% endif %}
 </main>
diff --git a/party/tests/test_tegn.py b/party/tests/test_tegn.py
index d18117d..10607b1 100644
--- a/party/tests/test_tegn.py
+++ b/party/tests/test_tegn.py
@@ -203,3 +203,12 @@ class TegnViewTests(TestCase):
         self.assertIsInstance(rooms.get_room(code), TegnRoom)
         page = self.client.get("/host/", headers={"x-authentik-username": "agw"}, REMOTE_ADDR="127.0.0.1")
         self.assertContains(page, 'value="tegn"')
+
+    def test_each_game_is_its_own_submit_button(self):
+        """Fejlen: spillet var en radio med Fup og Fakta afkrydset. En TV-fjernbetjening flytter fokus uden at
+        afkrydse, og Enter/OK på den fokuserede "Tegn og Gæt" sendte formularen med game=fof."""
+        page = self.client.get("/host/", headers={"x-authentik-username": "agw"}, REMOTE_ADDR="127.0.0.1")
+        html = page.content.decode()
+        self.assertNotRegex(html, r'<input[^>]*name="game"')
+        for key in ("fof", "tegn"):
+            self.assertRegex(html, rf'<button type="submit" name="game" value="{key}"')