3246 sujets

Critiques de vos sites: code et design

Avis et critique sur mon portfolio

salut
j'aime bien tes données en ligne de code.

dans les sections il n'y a presque pas de différence entre les 2 gris foncés
çà pourrait passer comme étant une erreur

upload/1676026136-47649-capturedaeacran2023-02-10aa11.png
Modifié par drphilgood (10 Feb 2023 - 11:49)
Oui j’ai pensé à augmenter légèrement le contraste des cards

Je ne pense pas que les personnes le voient comme une erreur Smiley ravi
Modérateur
Bonjour,

1) C'est beau et original !

2) Tous les indicateurs sont au vert avec "Lighthouse". Bravo !

3) Dans la section "QUI SUIS-JE ?", "ont commencés" ne devrait pas avoir de "s" me semble-t-il.

4) Dans la section "RÉALISATIONS", il ne faut pas mettre de majuscule à "(Projet étudiant)" . D'une manière générale, et contrairement à l'anglais, les majuscules doivent être utilisées avec parcimonie en français.

5) Il y a trop d'espace devant le texte du lien "ME CONTACTER".

6) Normalement le contenu d'une balise <section> devrait commencer par un <hn>. Et y avait-il besoin de toutes ces <div> ? Tout ceci rend un peu plus difficile les analyses automatiques par robot du contenu du site. Ceci étant, vu que tout le monde saupoudre les sites de <section> sans les commencer par un titre et inonde le code de <div>, ce n'est pas très grave ! Smiley lol

7) Dans <main role="main">, role="main" est inutile. D'une manière générale, on ne devrait mettre des rôles que si c'est nécessaire.

Amicalement,
Modifié par parsimonhi (20 Feb 2023 - 05:57)
parsimonhi a écrit :
role="main" est inutile. D'une manière générale, on ne devrait mettre des rôles que si c'est nécessaire.

Bonjour. Bien que les validateurs le considèrent souvent comme inutile, cet attribut est pourtant bien nécessaire pour les lecteurs d'écrans selon les normes d'accessibilité du moment. Alors oui, c'est redondant avec la balise ad hoc, mais pour l'instant c'est encore d'actualité.
Modifié par Olivier C (20 Feb 2023 - 16:54)
Modérateur
Bonjour,
Olivier C a écrit :

Bonjour. Bien que les validateurs le considèrent souvent comme inutile, cet attribut est pourtant bien nécessaire pour les lecteurs d'écrans selon les normes d'accessibilité du moment. Alors oui, c'est redondant avec la balise ad hoc, mais pour l'instant c'est encore d'actualité.

Pas trop d'accord avec ça car on est en 2023 et HTML5 a presque 10 ans. Mais tu as peut-être raison ! Smiley cligne

Amicalement,
Il faut bien comprendre qu'il existe une différence entre l'idéal :
"MDN" a écrit :
La première règle d'ARIA s'énonce ainsi : « Si vous pouvez utiliser un élément natif HTML ou un attribut avec la sémantique et le comportement voulu qui existe nativement, faites-le plutôt que d'utiliser un autre élément en lui ajoutant un rôle, un état ou une propriété ARIA afin de le rendre accessible. ».

... et la pratique. HTML5 a presque 10 ans, oui, mais les lecteurs d'écrans ne suivent pas. Déjà qu'ils n'arrivent même pas à suivre la norme ARIA qui pourtant est faite pour eux, certains d'entre eux ne la prennent en charge que très partiellement.

Nous devrions donc encore mettre les principaux rôles structurant une page web (navigation, main, banner, contentInfo...) et pour te faire crisser un peu plus des dents : un seul h1 par page reste toujours d'actualité dans ce contexte (et oui !).
Prévenir un modérateur
Répondre au sujet
Aller à