Skip to content
OverviewSolutions
2
Comments
2
pudding-shark
@pudding-shark

All comments

  • P
    galonaranjo•80
    @galonaranjo
    What are you most proud of, and what would you do differently next time?

    Getting my workflow down. I've been really diligent about thoroughly editing files and making sure everything is buttoned before committing. Had a lot of fun playing and understanding transitions (cubic beziers, anyone?) and hover states.

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

    Understanding placement of `

    Galo Social Links

    2
    pudding-shark•20
    @pudding-shark

    ,

  • pudding-shark•20
    @pudding-shark

    Hello, good work. I have a few suggestions.

    Keep in mind that I'm a newbie, so the suggestions I have are based on feedback others have given me.

    • Wrap the <conteiner> in a <main> tag. It helps screen readers identify the main section of the page. This will make it more accessible.
    • Using px for font-size causes big issues in terms of accessibility. Instead, use rem. Click here to know why.
    • Since all of the text have the same font, you only need to set font-family: "Outfit", sans-serif once, either on the body (the children will inherent the value) or * (this selects all elements). Same with text-align: center.

    Hope this helps.