Skip to content
OverviewSolutions
3
Comments
3
Alex.
@akelee

All comments

  • Alex.•30
    @akelee

    I like how you defined reusable color variables in your CSS :root

    Great solution with great accuracy :)

  • Alex.•30
    @akelee

    Hey!

    I noticed that you've imported multiple font families, which might not be necessary in this case (eg: Cairo, Figtree, Outfit, Inter). This could slow down the page loading speed, but no big issue.

    Also, maybe a bit more padding around the card image, but overall, it was a great solution you provided :)

  • Alex.•30
    @akelee

    Great solution!

    I like how you kept all styling in the style.css file by doing this:

    @import url('https://fonts.googleapis.com/css2?family=Outfit:wght@100..900&display=swap');

    vs how I did it in my index.html file:

    <link href="https://fonts.googleapis.com/css2?family=Outfit:wght@100..900&display=swap" rel="stylesheet">

    (both have pros and cons)