22-11-2019, 19:53:06
Désolé d'arriver un peu tard sur le sujet.
Voici mes remarques qui ne sont pas prioritaires vu que l'app fonctionne bien tel quel :
- mettre la même couleur de fond en haut et en bas. (header et footer)
- uniformiser les modales : celle du choix du niveau de difficulté ne ressemble pas (et devrait griser l'arrière plan) à celui de félicitation ou celui qui précise qu'il y a encore des erreurs.
- idéalement, si les wagons pouvaient être centrés verticalement dans la réserve et le train dans le paysage (id=receptacle)
- les boutons OK dans les modales sont assez ternes : ça donne le sentiment qu'ils sont grisés
J'ai d'autres remarque sur les modales mais j'ai cru comprendre que tu voulais avoir un paquet de composants réutilisables donc ça n'a peut-être pas sa place ici.
L'idéal serait que la modal ai le focus quand elle est ouverte et qu'on puisse naviguer au clavier.
Notamment, sortir de la modal avec la touche ECHAP.
Niveau code, j'ai parcouru en rapide.
Y'a de net améliorations donc "chapeau".
En revanche, quand tu mets ""use strict" en entête d'un fichier js, il faut le mettre partout si possible.
En réalité, les navigateurs ont 2 moteurs de javascript : 1 qui supporte l'ecmascript > 5 et un < 5.
Si tu ne peux pas faire autrement (migration d'un gros site de ecmascript < 5 vers supérieur), tu mixes les 2.
Hors, toi tu pars de zéro et en mettant "use strict" partout, tu vas forcer le navigateur a n'utiliser que le moteur le plus récent donc tu gagnes des ressources (RAM, CPU IO voir GPU).
Voici mes remarques qui ne sont pas prioritaires vu que l'app fonctionne bien tel quel :
- mettre la même couleur de fond en haut et en bas. (header et footer)
- uniformiser les modales : celle du choix du niveau de difficulté ne ressemble pas (et devrait griser l'arrière plan) à celui de félicitation ou celui qui précise qu'il y a encore des erreurs.
- idéalement, si les wagons pouvaient être centrés verticalement dans la réserve et le train dans le paysage (id=receptacle)
- les boutons OK dans les modales sont assez ternes : ça donne le sentiment qu'ils sont grisés
J'ai d'autres remarque sur les modales mais j'ai cru comprendre que tu voulais avoir un paquet de composants réutilisables donc ça n'a peut-être pas sa place ici.
L'idéal serait que la modal ai le focus quand elle est ouverte et qu'on puisse naviguer au clavier.
Notamment, sortir de la modal avec la touche ECHAP.
Niveau code, j'ai parcouru en rapide.
Y'a de net améliorations donc "chapeau".
En revanche, quand tu mets ""use strict" en entête d'un fichier js, il faut le mettre partout si possible.
En réalité, les navigateurs ont 2 moteurs de javascript : 1 qui supporte l'ecmascript > 5 et un < 5.
Si tu ne peux pas faire autrement (migration d'un gros site de ecmascript < 5 vers supérieur), tu mixes les 2.
Hors, toi tu pars de zéro et en mettant "use strict" partout, tu vas forcer le navigateur a n'utiliser que le moteur le plus récent donc tu gagnes des ressources (RAM, CPU IO voir GPU).