Skip to content
  • Unlock Pro
  • Log in with GitHub
Profile
OverviewSolutions
20
Comments
1
P
Treasure Kabareebe
@trekab

All comments

  • Yahir_AM•370
    @Yahir-am
    Submitted almost 3 years ago

    Product preview card component using HTML & CSS

    3
    P
    Treasure Kabareebe•290
    @trekab
    Posted almost 3 years ago

    Nicely done, @Yahir-am! I like your use of CSS variables. They make your CSS readable and more maintainable. However, here are some suggestions to make your page look even better:

    • The mobile version appears to have some excess bottom white spacing after the button. Clearing this would make your page look more like the design spec.
    • It would also be great to provide a fallback font whenever you specify a font family in your CSS. e.g. font-family: "Montserrat", sans-serif; would imply that sans-serif is the fallback font. I find the introduction of this article(CSS Fallback Fonts) insightful about fallback fonts.
    • Finally, writing mobile-first CSS is a nice practice to follow when building responsive web pages. Here is a great article for more insight.

    I hope you find this feedback beneficial

    Happy coding!

    Marked as helpful
Frontend Mentor logo

Stay up to datewith new challenges, featured solutions, selected articles, and our latest news

Frontend Mentor

  • Unlock Pro
  • Contact us
  • FAQs
  • Become a partner

Explore

  • Learning paths
  • Challenges
  • Solutions
  • Articles

Community

  • Discord
  • Guidelines

For companies

  • Hire developers
  • Train developers
© Frontend Mentor 2019 - 2025
  • Terms
  • Cookie Policy
  • Privacy Policy
  • License

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub