You can not select more than 25 topics
Topics must start with a letter or number, can include dashes ('-') and can be up to 35 characters long.
18 lines
739 B
18 lines
739 B
3 years ago
|
Les plus :
|
||
|
|
- lisibilité et modularité du CSS. Des classes, en complément ou en remplacement, pour le h1 et le p aurait été un parfait en terme de modularité.
|
||
|
|
- bonne indentation et exploitation du scss, commentaires sont un plus.
|
||
|
|
- menu responsive
|
||
|
|
|
||
|
|
|
||
|
|
Les moins :
|
||
|
|
- Un overlay aurait été un plus
|
||
|
|
- la popin tout en violet est un peu raide, surtout avec un texte noir
|
||
|
|
- certaines couleurs définies en variables ne sont pas utilisées.
|
||
|
|
- il aurait fallut fusionner icon-style et main pour éviter un appel réseau
|
||
|
|
- checkbox newsletter ko, checkbox popin ok
|
||
|
|
- footer non responsive
|
||
|
|
- type de l'input mail aurait du etre email
|
||
|
|
- pourquoi une div dans le footer ?
|
||
|
|
- manque de title, aria-
|
||
|
|
- pas de description ni keywords
|