From e7b5082398a6ea58b7c43449993f24dbe2627111 Mon Sep 17 00:00:00 2001
From: Davide Grilli <davide.grilli@outlook.com>
Date: Mon, 7 Sep 2026 13:15:17 +0200
Subject: [PATCH] Squadra: un solo controllo per il criterio della classifica
 interna

La select nativa e il bottone-drawer cambiavano lo stesso criterio in due modi
contemporaneamente, effetto ridondante e poco curato. Ora un'unica riga
tappabile mostra il criterio attivo e apre il drawer di scelta.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
---
 src/routes/squadra.tsx | 37 ++++++++++++-------------------------
 1 file changed, 12 insertions(+), 25 deletions(-)

diff --git a/src/routes/squadra.tsx b/src/routes/squadra.tsx
index 91b7161..d4e414c 100644
--- a/src/routes/squadra.tsx
+++ b/src/routes/squadra.tsx
@@ -1,8 +1,8 @@
 import { useState } from "react";
 import { createFileRoute } from "@tanstack/react-router";
-import { Cake, ChevronDown, Crown, SlidersHorizontal, Trophy } from "lucide-react";
+import { Cake, ChevronDown, Crown, Trophy } from "lucide-react";
 import { cn } from "@/lib/utils";
-import { PageHeader, Select, StatTile } from "@/components/crapp/ui-bits";
+import { PageHeader, StatTile } from "@/components/crapp/ui-bits";
 import { BarraSottosezioni } from "@/components/crapp/BarraSottosezioni";
 import { Avatar } from "@/components/crapp/Avatar";
 import { formatData } from "@/lib/crapp-data";
@@ -273,32 +273,19 @@ function Squadra() {
             aTuttoLarghezza: true,
             contenuto: (
               <>
-                <div className="flex min-w-0 items-center gap-2 border-b border-border bg-card px-5 py-2.5">
+                <button
+                  type="button"
+                  onClick={() => setFiltroAperto(true)}
+                  aria-haspopup="dialog"
+                  className="flex w-full min-w-0 items-center gap-2 border-b border-border bg-card px-5 py-2.5 text-left"
+                >
                   <Trophy className="h-4 w-4 shrink-0 text-warning" aria-hidden />
                   <span className="shrink-0 text-sm text-foreground/80">Classifica per</span>
-                  <span className="min-w-0 flex-1">
-                    <Select
-                      value={criterio}
-                      onChange={(e) => setCriterio(e.target.value as Criterio)}
-                      aria-label="Criterio della classifica interna"
-                      className="h-9 rounded-lg border-border bg-background font-semibold shadow-none"
-                    >
-                      {criteri.map((c) => (
-                        <option key={c.id} value={c.id}>
-                          {c.label}
-                        </option>
-                      ))}
-                    </Select>
+                  <span className="min-w-0 flex-1 truncate text-right text-sm font-bold">
+                    {criteri.find((c) => c.id === criterio)?.label}
                   </span>
-                  <button
-                    type="button"
-                    onClick={() => setFiltroAperto(true)}
-                    className="grid h-9 w-9 shrink-0 place-items-center rounded-lg border border-border text-muted-foreground"
-                    aria-label="Scegli criterio classifica"
-                  >
-                    <SlidersHorizontal className="h-4 w-4" />
-                  </button>
-                </div>
+                  <ChevronDown className="h-4 w-4 shrink-0 text-muted-foreground" aria-hidden />
+                </button>
 
                 <Drawer open={filtroAperto} onOpenChange={setFiltroAperto}>
                   <DrawerContent>