Mejorar el espaciado y la organización de la interfaz en el componente MapaCurricular

src/routes/planes/$planId/_detalle/mapa.tsx

@@ -1,13 +1,8 @@
 /* eslint-disable jsx-a11y/no-static-element-interactions */
 /* eslint-disable jsx-a11y/label-has-associated-control */
 import { createFileRoute } from '@tanstack/react-router'
-import {
-  Plus,
-  ChevronDown,
-  AlertTriangle,
-  Trash2,
-  Pencil,
-} from 'lucide-react'
+import { Plus, ChevronDown, AlertTriangle, Trash2, Pencil } from 'lucide-react'
+import * as Icons from 'lucide-react'
 import { useMemo, useState, useEffect, Fragment } from 'react'
 
 import type { TipoAsignatura } from '@/data'
@@ -36,6 +31,12 @@ import {
   SelectTrigger,
   SelectValue,
 } from '@/components/ui/select'
+import {
+  Tooltip,
+  TooltipContent,
+  TooltipProvider,
+  TooltipTrigger,
+} from '@/components/ui/tooltip'
 import {
   useCreateLinea,
   useDeleteLinea,
@@ -45,12 +46,6 @@ import {
   useUpdateAsignatura,
   useUpdateLinea,
 } from '@/data'
-import {
-  Tooltip,
-  TooltipContent,
-  TooltipProvider,
-  TooltipTrigger,
-} from '@/components/ui/tooltip'
 
 // --- Mapeadores (Fuera del componente para mayor limpieza) ---
 const palette = [
@@ -137,8 +132,6 @@ function StatItem({
   )
 }
 
-import * as Icons from 'lucide-react'
-
 const estadoConfig: Record<
   Asignatura['estado'],
   {
@@ -210,10 +203,10 @@ function AsignaturaCardItem({
             onDragStart={(e) => onDragStart(e, asignatura.id)}
             onClick={onClick}
             className={[
-              'group relative h-[200px] w-[272px] shrink-0 overflow-hidden rounded-[22px] border text-left',
+              'group relative h-50 w-50 shrink-0 overflow-hidden rounded-[22px] border text-left',
               'transition-all duration-300 ease-out',
-              'focus-visible:outline-none focus-visible:ring-2 focus-visible:ring-ring/30',
-              'active:cursor-grabbing cursor-grab',
+              'focus-visible:ring-ring/30 focus-visible:ring-2 focus-visible:outline-none',
+              'cursor-grab active:cursor-grabbing',
               isDragging
                 ? 'scale-[0.985] opacity-45 shadow-none'
                 : 'hover:-translate-y-1 hover:shadow-lg',
@@ -235,7 +228,7 @@ function AsignaturaCardItem({
 
             {/* glow decorativo */}
             <div
-              className="absolute -top-10 -right-10 h-28 w-28 rounded-full blur-2xl transition-transform duration-500 group-hover:scale-110"
+              className="absolute -top-10 -right-10 h-28 w-28 rounded-full blur-2xl"
               style={{ backgroundColor: hexToRgba(lineaColor, 0.22) }}
             />
 
@@ -243,7 +236,7 @@ function AsignaturaCardItem({
               {/* top */}
               <div className="flex items-start justify-between gap-2">
                 <div
-                  className="inline-flex h-8 max-w-[200px] items-center gap-1.5 rounded-full border px-2.5 text-[11px] font-semibold"
+                  className="inline-flex h-8 max-w-32 items-center gap-1.5 rounded-full border px-2.5 text-[11px] font-semibold"
                   style={{
                     borderColor: hexToRgba(lineaColor, 0.2),
                     backgroundColor: hexToRgba(lineaColor, 0.1),
@@ -251,37 +244,29 @@ function AsignaturaCardItem({
                   }}
                 >
                   <Icons.KeyRound className="h-3.5 w-3.5 shrink-0" />
-                  <span className="truncate">{asignatura.clave || 'Sin clave'}</span>
-                </div>
-
-                <div className="relative flex h-8 items-center overflow-hidden rounded-full bg-background/70 px-2 backdrop-blur-sm">
-                  <div className="flex gap-4 items-center gap-1.5 transition-transform duration-300 group-hover:-translate-x-[72px]">
-                    <span className={`h-2.5 w-2.5 rounded-full ${estado.dot}`} />
-                    <EstadoIcon
-                      className={[
-                        'h-3.5 w-3.5 text-foreground/65',
-                        asignatura.estado === 'generando' ? 'animate-spin' : '',
-                      ].join(' ')}
-                    />
-                  </div>
-                  <div
-                    className={[
-                      'absolute right-2 flex translate-x-6 items-center gap-1.5 opacity-0 transition-all duration-300',
-                      'group-hover:translate-x-0 group-hover:opacity-100'
-                    ].join(' ')}
-                  >
-                    <span className="text-[11px] font-semibold whitespace-nowrap">
-                      {estado.label}
+                  <span className="truncate">
+                    {asignatura.clave || 'Sin clave'}
                   </span>
                 </div>
 
+                <Tooltip>
+                  <TooltipTrigger asChild>
+                    <div className="bg-background/70 flex h-8 items-center rounded-full px-2 backdrop-blur-sm">
+                      <EstadoIcon className="text-foreground/65 h-3.5 w-3.5" />
                     </div>
+                  </TooltipTrigger>
+                  <TooltipContent side="left">
+                    <span className="text-xs font-semibold">
+                      {estado.label}
+                    </span>
+                  </TooltipContent>
+                </Tooltip>
               </div>
 
               {/* titulo */}
-              <div className="mt-4 min-h-[72px]">
+              <div className="mt-4 min-h-18">
                 <h3
-                  className="overflow-hidden text-[18px] leading-[1.08] font-bold text-foreground"
+                  className="text-foreground overflow-hidden text-[18px] leading-[1.08] font-bold"
                   style={{
                     display: '-webkit-box',
                     WebkitLineClamp: 3,
@@ -295,45 +280,45 @@ function AsignaturaCardItem({
               {/* bottom */}
               <div className="mt-auto grid grid-cols-3 gap-2">
                 <div className="rounded-2xl border border-white/40 bg-white/55 px-2.5 py-2 backdrop-blur-sm dark:border-white/10 dark:bg-white/5">
-                  <div className="mb-1 flex items-center gap-1.5 text-muted-foreground">
+                  <div className="text-muted-foreground mb-1 flex items-center gap-1.5">
                     <Icons.Award className="h-3.5 w-3.5" />
-                    <span className="text-[10px] font-medium uppercase tracking-wide">
+                    <span className="text-[10px] font-medium tracking-wide uppercase">
                       CR
                     </span>
                   </div>
-                  <div className="text-sm font-bold text-foreground">
+                  <div className="text-foreground text-sm font-bold">
                     {asignatura.creditos}
                   </div>
                 </div>
 
                 <div className="rounded-2xl border border-white/40 bg-white/55 px-2.5 py-2 backdrop-blur-sm dark:border-white/10 dark:bg-white/5">
-                  <div className="mb-1 flex items-center gap-1.5 text-muted-foreground">
+                  <div className="text-muted-foreground mb-1 flex items-center gap-1.5">
                     <Icons.Clock3 className="h-3.5 w-3.5" />
-                    <span className="text-[10px] font-medium uppercase tracking-wide">
+                    <span className="text-[10px] font-medium tracking-wide uppercase">
                       HD
                     </span>
                   </div>
-                  <div className="text-sm font-bold text-foreground">
+                  <div className="text-foreground text-sm font-bold">
                     {asignatura.hd}
                   </div>
                 </div>
 
                 <div className="rounded-2xl border border-white/40 bg-white/55 px-2.5 py-2 backdrop-blur-sm dark:border-white/10 dark:bg-white/5">
-                  <div className="mb-1 flex items-center gap-1.5 text-muted-foreground">
+                  <div className="text-muted-foreground mb-1 flex items-center gap-1.5">
                     <Icons.BookOpenText className="h-3.5 w-3.5" />
-                    <span className="text-[10px] font-medium uppercase tracking-wide">
+                    <span className="text-[10px] font-medium tracking-wide uppercase">
                       HI
                     </span>
                   </div>
-                  <div className="text-sm font-bold text-foreground">
+                  <div className="text-foreground text-sm font-bold">
                     {asignatura.hi}
                   </div>
                 </div>
               </div>
 
               {/* drag affordance */}
-              <div className="pointer-events-none absolute right-3 bottom-3 rounded-full bg-background/70 p-1.5 opacity-0 backdrop-blur-sm transition-all duration-300 group-hover:opacity-100">
-                <Icons.GripVertical className="h-4 w-4 text-muted-foreground/55" />
+              <div className="bg-background/70 pointer-events-none absolute right-3 bottom-3 rounded-full p-1.5 opacity-0 backdrop-blur-sm transition-all duration-300 group-hover:opacity-100">
+                <Icons.GripVertical className="text-muted-foreground/55 h-4 w-4" />
               </div>
             </div>
           </button>
@@ -341,7 +326,13 @@ function AsignaturaCardItem({
 
         <TooltipContent side="bottom">
           <div className="text-xs">
-            {lineaNombre ? `${lineaNombre} · ` : ''}
+            {/* ciclo */}
+            {asignatura.ciclo ? (
+              <span className="font-bold">C{asignatura.ciclo} · </span>
+            ) : null}
+            {lineaNombre ? (
+              <span className="font-medium">{lineaNombre} · </span>
+            ) : null}
             {asignatura.nombre}
           </div>
         </TooltipContent>
@@ -689,7 +680,7 @@ function MapaCurricularPage() {
     return <div className="p-10 text-center">Cargando mapa curricular...</div>
 
   return (
-    <div className="container mx-auto px-2 py-6">
+    <div className="container">
       {/* Header */}
       <div className="mb-6 flex items-center justify-between">
         <div>
@@ -768,11 +759,10 @@ function MapaCurricularPage() {
       </div>
 
       <div className="overflow-x-auto pb-6">
-        <div className="min-w-[1500px]">
         <div
           className="grid gap-3"
           style={{
-              gridTemplateColumns: `220px repeat(${ciclosTotales}, minmax(auto, 1fr)) 120px`,
+            gridTemplateColumns: `140px repeat(${ciclosTotales}, minmax(auto, 1fr)) 120px`,
           }}
         >
           <div className="self-end px-2 text-xs font-bold text-slate-400">
@@ -798,10 +788,11 @@ function MapaCurricularPage() {
             return (
               <Fragment key={linea.id}>
                 <div
-                    className={`group relative flex items-center justify-between rounded-xl border-l-4 p-4 transition-all ${lineColors[idx % lineColors.length]
+                  className={`group relative flex items-center justify-between rounded-xl border-l-4 p-3 transition-all ${
+                    lineColors[idx % lineColors.length]
                   } ${editingLineaId === linea.id ? 'bg-white ring-2 ring-teal-500/20' : ''}`}
                 >
-                    <div className="flex-1 overflow-hidden">
+                  <div className="min-w-0 flex-1 overflow-hidden">
                     <span
                       contentEditable={editingLineaId === linea.id}
                       suppressContentEditableWarning
@@ -814,7 +805,8 @@ function MapaCurricularPage() {
                           setTempNombreLinea(linea.nombre)
                         }
                       }}
-                        className={`block w-full text-xs font-bold break-words outline-none ${editingLineaId === linea.id
+                      className={`block w-full truncate text-xs font-bold break-words outline-none ${
+                        editingLineaId === linea.id
                           ? 'cursor-text border-b border-teal-500/50 pb-1'
                           : 'cursor-pointer'
                       }`}
@@ -822,7 +814,7 @@ function MapaCurricularPage() {
                       {linea.nombre}
                     </span>
                   </div>
-                    <div className="flex items-center gap-2">
+                  <div className="ml-2 flex flex-shrink-0 items-center gap-1">
                     <button
                       onClick={() => setEditingLineaId(linea.id)}
                       className="..."
@@ -848,8 +840,7 @@ function MapaCurricularPage() {
                     {asignaturas
                       .filter(
                         (m) =>
-                            m.ciclo === ciclo &&
-                            m.lineaCurricularId === linea.id,
+                          m.ciclo === ciclo && m.lineaCurricularId === linea.id,
                       )
                       .map((m) => (
                         <AsignaturaCardItem
@@ -904,36 +895,35 @@ function MapaCurricularPage() {
           </div>
         </div>
       </div>
-      </div>
 
       {/* Asignaturas Sin Asignar */}
-      <div className="mt-12 rounded-[28px] border border-border bg-card/80 p-5 shadow-sm backdrop-blur-sm">
+      <div className="border-border bg-card/80 mt-12 rounded-[28px] border p-5 shadow-sm backdrop-blur-sm">
         <div className="mb-5 flex flex-col gap-3 md:flex-row md:items-center md:justify-between">
           <div className="min-w-0">
             <div className="flex items-center gap-2">
-              <div className="flex h-9 w-9 items-center justify-center rounded-2xl bg-muted text-muted-foreground">
+              <div className="bg-muted text-muted-foreground flex h-9 w-9 items-center justify-center rounded-2xl">
                 <Icons.Inbox className="h-4.5 w-4.5" />
               </div>
 
               <div className="min-w-0">
                 <div className="flex items-center gap-2">
-                  <h3 className="text-sm font-bold tracking-wide text-foreground uppercase">
+                  <h3 className="text-foreground text-sm font-bold tracking-wide uppercase">
                     Bandeja de entrada
                   </h3>
 
-                  <div className="inline-flex h-6 min-w-6 items-center justify-center rounded-full bg-muted px-2 text-[11px] font-semibold text-muted-foreground">
+                  <div className="bg-muted text-muted-foreground inline-flex h-6 min-w-6 items-center justify-center rounded-full px-2 text-[11px] font-semibold">
                     {unassignedAsignaturas.length}
                   </div>
                 </div>
 
-                <p className="mt-0.5 text-sm text-muted-foreground">
+                <p className="text-muted-foreground mt-0.5 text-sm">
                   Asignaturas sin ciclo o línea curricular
                 </p>
               </div>
             </div>
           </div>
 
-          <div className="flex items-center gap-2 rounded-full border border-dashed border-border bg-background/80 px-3 py-1.5 text-xs text-muted-foreground">
+          <div className="border-border bg-background/80 text-muted-foreground flex items-center gap-2 rounded-full border border-dashed px-3 py-1.5 text-xs">
             <Icons.MoveDown className="h-3.5 w-3.5" />
             <span>Arrastra aquí para desasignar</span>
           </div>
@@ -969,18 +959,18 @@ function MapaCurricularPage() {
               ))}
             </div>
           ) : (
-            <div className="flex min-h-[188px] flex-col items-center justify-center rounded-[20px] border border-border/70 bg-background/70 px-6 text-center">
-              <div className="mb-3 flex h-12 w-12 items-center justify-center rounded-2xl bg-muted text-muted-foreground">
+            <div className="border-border/70 bg-background/70 flex min-h-[188px] flex-col items-center justify-center rounded-[20px] border px-6 text-center">
+              <div className="bg-muted text-muted-foreground mb-3 flex h-12 w-12 items-center justify-center rounded-2xl">
                 <Icons.CheckCheck className="h-5 w-5" />
               </div>
 
-              <p className="text-sm font-semibold text-foreground">
+              <p className="text-foreground text-sm font-semibold">
                 No hay asignaturas pendientes
               </p>
 
-              <p className="mt-1 max-w-md text-sm text-muted-foreground">
-                Todo está colocado en el mapa. Arrastra una asignatura aquí para quitarle
-                ciclo y línea curricular.
+              <p className="text-muted-foreground mt-1 max-w-md text-sm">
+                Todo está colocado en el mapa. Arrastra una asignatura aquí para
+                quitarle ciclo y línea curricular.
               </p>
             </div>
           )}