thcolin / chuck-norrit Goto Github PK

Salut Thomas 👋 😄
Merci d'avoir pris le temps de réaliser ce test. On a analysé ton code avec attention et on a bien kiffé!
Globalement, voilà ce qu'on a retenu:

## Points positifs ✅

• Utilisation de git
• Présence d'un .gitignore
• Présence d'un readme avec des instructions d'installation et de lancement
• package.json avec toutes les dépendances
• Présence d'un package-lock
• Présence d'un transpileur configuré (Babel)
• Utilisation d'une lib front JS obscure (:smile:) et de son écosystème
• Présence d'un polyfill pour fetch() 👌
• Présence de tests
• Tests qui passents (:clap:)
• Lib pour gérer l'état de l'application (pattern Observer avec Emitter)
• Pas de bazooka Redux 👍
• Pas d'utilisation du bazooka Sass (vu le test c'était pas utile de s'embarasser avec ça, donc bien vu)
• Présence d'un reset (Normalize.css)
• UI simple et clean 👍 ✨
• App très, très, très légère ❤️
• Code straightforward, pas de sur-conception
• Emojis, emojis everywhere ✨
• Last but not least, l'app fonctionne 🎉

Points à revoir ❓

• Ta structure de fichier. Un petit dossier src, rien que pour séparer tes fichiers de config, ça aurait été un peu mieux quand même 😉
• Pas d'ES6, alors que tu as un .babelrc, et c'est bien dommage, ton code aurait pu être encore plus concis et direct qu'il ne l'est déjà 😕

Points à revoir mais pas essentiels ⚠️

• Avoir un linter JS, ça aurait été cool (même si vu le nombre de ligne de code, c'est pas essentiel)
• En CSS, il vaut mieux éviter de faire ça #main ul li (je sais pas si tu es familier avec le concept de spécificité en CSS donc je te conseille cet article pour en apprendre un peu plus. Ici c'est tellement léger que tu aurais pu cibler directement tes li
• Evite aussi de mettre tous tes sélecteurs sur une ligne comme ici ⬇️

#main form input[type="search"]:focus,#main form button[type="submit"]:focus, #main form button[type="submit"]:active {

}

La bonne pratique, c'est d'avoir un retour à la ligne 😉

• Evite aussi de faire un outline: 0. Tu arrives à peu près à compenser avec un effet au focus, mais il est très subtil, c'est pas ouf niveau accessibilité

Et voilà! En tout cas on vraiment apprécié ton travail, ta curiosité et ta détermination. On connaissait pas les libs que tu as utilisé, c'est une bonne découverte!
Globalement c'est cool donc, bravo à toi 😄 🏆