Italian variations

Author: TonNom

src/pages/opening.tsx

@@ -1,23 +1,19 @@
+import { italianGameVariations } from "../data/openings to learn/italian";
 import { Box, Typography, Button, Stack } from "@mui/material";
-import { useState, useMemo, useEffect, useCallback } from "react";
+import { useState, useMemo, useEffect } from "react";
 import Board from "../components/board";
 import { Chess } from "chess.js";
 import { atom, useAtom } from "jotai";
 import { useChessActions } from "../hooks/useChessActions";
 import { Color } from "../types/enums";
 import { CurrentPosition } from "../types/eval";
-import { Variation, italianGameVariations } from "../data/openings to learn/italian";
+import type { Variation } from "../data/openings to learn/italian";
 import OpeningProgress from "../components/OpeningProgress";
 
-// Extension du type Variation pour inclure les propriétés optionnelles
-interface ExtendedVariation extends Variation {
-  fen?: string;
-  color?: Color;
-}
-
 // Détermine la couleur d'apprentissage pour la variante (par défaut blanc, mais extensible)
-function getLearningColor(variation: ExtendedVariation): Color {
-  return variation.color || Color.White;
+function getLearningColor(variation: Variation): Color {
+  // TODO: utiliser variation.color si défini, sinon blanc
+  return Color.White;
 }
 
 export default function OpeningPage() {
@@ -121,11 +117,7 @@ export default function OpeningPage() {
       for (let i = 0; i < moveIdx; i++) expectedMove.move(selectedVariation.moves[i]);
       const expected = expectedMove.move(selectedVariation.moves[moveIdx]);
       if (!expected || last.from !== expected.from || last.to !== expected.to) {
-        // Mauvais coup : afficher l'icône de blunder
-        setShowBlunder({ show: true, square: last.to });
-        setTimeout(() => setShowBlunder({ show: false, square: null }), 1500);
-        
-        // Annuler et annoter
+        // Mauvais coup : annuler et annoter
         let mistakeType = "Mistake";
         if (last.captured || last.san.includes("#")) mistakeType = "Blunder";
         setLastMistake({ from: last.from, to: last.to, type: mistakeType });
@@ -183,9 +175,6 @@ export default function OpeningPage() {
   const openingKey = "italian";
   const progressMode = trainingMode ? "training" : "learning";
   const progressStorageKey = `${openingKey}-progress-${progressMode}`;
-  // État pour gérer l'affichage de l'icône de blunder
-  const [showBlunder, setShowBlunder] = useState<{show: boolean, square: string | null}>({show: false, square: null});
-
   const [completedVariations, setCompletedVariations] = useState<number[]>(() => {
     if (typeof window === "undefined") return [];
     try {
@@ -209,59 +198,11 @@ export default function OpeningPage() {
     // eslint-disable-next-line react-hooks/exhaustive-deps
   }, [moveIdx, selectedVariation, currentVariantIdx, progressMode]);
 
-  // Réinitialisation de la progression et de l'état du jeu
-  const handleResetProgress = useCallback(() => {
-    try {
-      // 1. Réinitialiser la progression dans le stockage local
-      localStorage.removeItem(progressStorageKey);
-      localStorage.setItem(progressStorageKey, JSON.stringify([]));
-      
-      // 2. Réinitialiser les états locaux
-      setCompletedVariations([]);
-      setTrainingMode(false);
-      setLastMistake(null);
-      
-      // 3. Mettre à jour l'index de la variante et le coup actuel
-      // D'abord, réinitialiser l'index de la variante pour forcer un rechargement
-      setCurrentVariantIdx(-1);
-      
-      // 4. Réinitialiser l'index du coup actuel
-      setMoveIdx(0);
-      
-      // 5. Créer une nouvelle instance de jeu
-      const newGame = new Chess();
-      
-      // 6. Mettre à jour l'état du jeu
-      setGame(newGame);
-      
-      // 7. Forcer un rafraîchissement de l'interface utilisateur
-      // Utiliser un délai pour s'assurer que les états précédents sont bien mis à jour
-      setTimeout(() => {
-        try {
-          // Forcer un nouveau rendu
-          window.dispatchEvent(new Event('resize'));
-          
-          // Mettre à jour l'index de la variante pour déclencher la réinitialisation du jeu
-          // Cela va déclencher l'effet qui met à jour le jeu avec la première variante
-          setCurrentVariantIdx(0);
-          
-          // S'assurer que le jeu est bien réinitialisé avec la première variante
-          if (variations.length > 0) {
-            const initialGame = new Chess();
-            // Appliquer les coups de la première variante si nécessaire
-            // (le premier coup sera appliqué automatiquement par l'effet)
-            setGame(initialGame);
-          }
-          
-        } catch (error) {
-          console.error("Erreur lors de la mise à jour de l'interface utilisateur:", error);
-        }
-      }, 50);
-      
-    } catch (error) {
-      console.error("Erreur lors de la réinitialisation de la progression:", error);
-    }
-  }, [progressStorageKey, setGame, variations.length]);
+  // Réinitialisation de la progression
+  const handleResetProgress = () => {
+    localStorage.removeItem(progressStorageKey);
+    setCompletedVariations([]);
+  };
 
   // Détermination de la case cible du dernier coup joué (pour overlay)
   const lastMoveSquare = useMemo(() => {
@@ -329,24 +270,7 @@ export default function OpeningPage() {
         </Box>
         {/* Zone de droite : échiquier responsive */}
         <Box sx={{ flex: 2, display: 'flex', alignItems: 'center', justifyContent: 'center', minWidth: 0 }}>
-          {allDone ? (
-            <Box sx={{ textAlign: 'center', p: 3 }}>
-              <Typography variant="h5" color="success.main" gutterBottom>
-                Félicitations !
-              </Typography>
-              <Typography variant="body1" sx={{ mb: 2 }}>
-                Vous avez terminé toutes les variantes de cette ouverture.
-              </Typography>
-              <Button 
-                variant="contained" 
-                color="primary" 
-                onClick={handleResetProgress}
-                sx={{ mt: 2 }}
-              >
-                Recommencer depuis le début
-              </Button>
-            </Box>
-          ) : selectedVariation && game ? (
+          {selectedVariation && !allDone && game && (
             <Box sx={{ width: '100%', maxWidth: 600, aspectRatio: '1', minWidth: { xs: 260, sm: 340, md: 400 }, mx: 'auto', position: 'relative' }}>
               <Board
                 id="LearningBoard"
@@ -362,7 +286,7 @@ export default function OpeningPage() {
                 trainingFeedback={trainingFeedback && lastMoveSquare ? { square: lastMoveSquare, ...trainingFeedback } : undefined}
               />
             </Box>
-          ) : null}
+          )}
         </Box>
       </Stack>
     </Box>