Skip to content
OverviewSolutions
5
Comments
3
Alrn
@Annabelle-Lrn

All comments

  • Alrn•50
    @Annabelle-LrnPosted 17 days ago

    Good job!

    0
  • zh4r•80
    @zh4rSubmitted 7 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
    Alrn•50
    @Annabelle-LrnPosted about 2 months 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
  • Alrn•50
    @Annabelle-LrnPosted about 2 months 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