From 6286c18a72f941716e8b3faa8591d0a191fa0165 Mon Sep 17 00:00:00 2001
From: Davide Grilli <davide.grilli@outlook.com>
Date: Thu, 3 Sep 2026 14:17:30 +0200
Subject: [PATCH] Sostituisce l'input libero del ruolo con un menu a tendina

Evita ruoli scritti in modo incoerente (maiuscole, refusi) sia in
modifica che in aggiunta giocatore.
---
 src/lib/giocatori-squadra.ts |  3 +++
 src/routes/admin.tsx         | 25 +++++++++++++++++++------
 2 files changed, 22 insertions(+), 6 deletions(-)

diff --git a/src/lib/giocatori-squadra.ts b/src/lib/giocatori-squadra.ts
index 8775769..92f2165 100644
--- a/src/lib/giocatori-squadra.ts
+++ b/src/lib/giocatori-squadra.ts
@@ -33,6 +33,9 @@ export type RigaGiocatoreSquadra = {
   data_tessera: string | null;
 };
 
+/** Ruoli ammessi in campo (pallavolo): usati per il menu a tendina del profilo squadra. */
+export const RUOLI = ["Palleggiatore", "Banda", "Opposto", "Centrale", "Libero", "Jolly"] as const;
+
 export const SQUADRA_KEY = ["giocatori-squadra"] as const;
 
 /** Rosa di riserva quando il database non risponde o non è ancora popolato. */
diff --git a/src/routes/admin.tsx b/src/routes/admin.tsx
index 2d2c400..2049a5a 100644
--- a/src/routes/admin.tsx
+++ b/src/routes/admin.tsx
@@ -22,6 +22,7 @@ import {
   nomeCompleto,
   numeroGiaUsato,
   prossimoIdGiocatore,
+  RUOLI,
   useAggiungiGiocatore,
   useGiocatoriSquadra,
   useImpostaAttivo,
@@ -223,12 +224,18 @@ function ModificaGiocatore({ g, profilo }: { g: GiocatoreSquadra; profilo: Profi
           />
         </Campo>
         <Campo label="Ruolo">
-          <input
+          <select
             value={squadraCorrente.ruolo}
-            maxLength={30}
             onChange={(e) => setDatiSquadra({ ...squadraCorrente, ruolo: e.target.value })}
             className={classiInput}
-          />
+          >
+            <option value="">—</option>
+            {RUOLI.map((r) => (
+              <option key={r} value={r}>
+                {r}
+              </option>
+            ))}
+          </select>
         </Campo>
       </div>
       <Campo label="Email (collegamento automatico al login, DD-018)">
@@ -540,12 +547,18 @@ function AggiungiGiocatore({ righe }: { righe: GiocatoreSquadra[] }) {
           />
         </Campo>
         <Campo label="Ruolo">
-          <input
+          <select
             value={dati.ruolo}
-            maxLength={30}
             onChange={(e) => setDati({ ...dati, ruolo: e.target.value })}
             className={classiInput}
-          />
+          >
+            <option value="">—</option>
+            {RUOLI.map((r) => (
+              <option key={r} value={r}>
+                {r}
+              </option>
+            ))}
+          </select>
         </Campo>
       </div>
       <Campo label="Email (collegamento automatico al login, opzionale)">