Skip to content
Submitted over 2 years ago

SASS, GIT, BEM, RWD

Sebastian•220
@SebastianLaka
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

Hi it's my third project and it learned me so much. If you know how make it better tell me please. Thank you :)

Community feedback

  • Oshu•630
    @oshudevPosted over 2 years ago

    Hi Sebastian, great job!

    Add the main tag and wrap the card for Accessibility. It should be like this:

      <body>
    <main>
      <div class="card"></div>
    </main>
  </body>

    Also when styling a button always put font-family: inherit; so that it matches the overall font family.

    Overall you did well! Hope it helps.

    Marked as helpful
    1
    Sebastian•220
    @SebastianLakaPosted over 2 years ago

    @eurus1108 Hi thank you for your feedback :) i changed and added main for html and added class for button as you said. It's helpful :D

    0

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