Hello,
Fais attention aussi à la propreté et la lisibilité du code : indentation inexistante, JavaScript à séparer du HTML, attributs style
, etc. C'est bien de proposer un code tout fait, encore faudrait-il qu'il soit facilement compréhensible et modifiable : les lecteurs n'auront pas forcément envie d'avoir le même rendu, ils souhaiteront sûrement adapter pour leur charte graphique.
Sinon le rendu est pas tout à fait accessible : certains éléments manquent de retour au focus
par exemple, d'autres changent complètement de style par moment (le slider horizontal notamment).
Au passage, dans ton, JS, évite innerHTML
quand tu veux seulement afficher du texte pur (sans HTML donc) : préfère textContent
ou insère un nœud texte avec les méthodes du DOM. Et pense à rester cohérent au niveau des quotes pour tes chaînes de caractères : choisis un style et garde-le dans tout le code, ne fais pas de mélanges.
En passant, pense aussi au responsive et autres : évite les tailles en pixels (de nos jours on utilise du em
ou du rem
) ou les typos forcées (un simple font-family: inherit
sera beaucoup plus utile et portable).
Un dernier point important aussi : n'oublie pas la compatibilité navigateur. Si ton code nécessite une version minimale indique-le ! Et s'il y a moyen d'augmenter la compatibilité via des polyfills ou autres hacks, ça peut être utile de les mentionner…