Skip to content
Submitted over 3 years ago

HTML CSS SASS VSCODE GITHUB Task 3

Bhupender Singh•100
@bunny1351
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...3-column preview card component
  • HTML
  • CSS
1newbie

Design comparison

SolutionDesign

Solution retrospective

Hey everyone

Please check my solution and provide suggestions.

Thank you

Community feedback

  • darryncodes•6,370
    @darryncodesPosted over 3 years ago

    Hi Bhupender,

    Excellent solution, well done.

    Some considerations:

    • use semantic html. your .container <div> could be <main> and your .attribution <div> could be <footer>. This should resolve the warnings in the accessibility report
    • I usually like to add transition: ease-in .3s; to buttons. Add it to your .learn styles. It'll make the hover affect less abrupt.
    Marked as helpful
    1
    Bhupender Singh•100
    @bunny1351Posted over 3 years ago

    @darryncodes Thank you for the suggestions

    0
  • Patrick•14,265
    @palgrammingPosted over 3 years ago

    You should remove that min width out of your media query

    1

Please log in to post a comment

Log in with GitHub

Join our Discord community

Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!

Join our Discord