All comments

• 

  zh4r• 50

  @zh4r

  Submitted 5 months ago

  What are you most proud of, and what would you do differently next time?

  • I really like how it turned out visually - I'm having a lot of fun with animations.
  • I went for mobile first this time, and the workflow feels better, I'll continue doing so.

  What challenges did you encounter, and how did you overcome them?

  • Semantic HTML - still not sure when to use what, and I don't think I've really overcome it...

  What specific areas of your project would you like help with?

  • Accessibility review would be appreciated.
  • HTML Semantics - I'm not sure if I used too much, or if I got it wrong...
  • SCSS

  

  Semantic HTML5 + SCSS | Animations

  1

  50

  

  Alrn• 40

  @Annabelle-Lrn

  Posted 4 days ago

  Beau travail, les effets au survol sont top entre le trait de soulignement et l'ombre de la boite qui augmente 😊.

  Etant novice mes seules suggestions concernant la sémantique sont :

  • il ne peut y avoir de balise <h2> sans balise <h1>

  • <article> est censé représenter un élément autonome , ce qui n'est pas le cas ici (sauf erreur)

  une petite coquille c'est glissée tu as 2 balises footer une avant la fermeture de la balise main et une après.

  Concernant le style :

  • Pour les unités (largeur et hauteur) utilises des unités relatives (REM) plutôt qu'absolue (px).

  j'espère que ce commentaire te seras utile. Bonne continuation

  0
• 

  leviipope• 40

  @leviipope

  Submitted 4 days ago

  

  Simple QR Code Website

  1

  20

  

  Alrn• 40

  @Annabelle-Lrn

  Posted 4 days ago

  Beau travail. Je suis débutante mais si je peux me permettre une ou deux suggestions :

  • Pour les unités utilisé des unités relatives (REM) plutôt qu'absolue (px). Et pour les images des %. cela permet de rendre le design responsive.

  • utiliser des classes plutôt que des id. L' id est unique contrairement à la classe. Par exemple pour les paragraphes ils ont des propriétés communes plutôt que les définir 2 fois avec une classe commune pour les deux permet de maintenir plus facilement le code. par ex :

  <p class="text t1">Improve your front-end skills by building projects</p>

  <p class="text t2">Scan the QR code to visit Frontend Mentor and take your coding skills to the next level</p>

  J'espère que ce commentaire sera utile.

  Bonne continuation 😊

  0