Skip to content
Submitted over 2 years ago

Responsive three column using flex

Muhamad Rizky•140
@bodashideung
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, this is my first time for submitting the solution in this site. I hope there is someone who want to take a look to my solution and give me some an advices. Thank you.

Community feedback

  • teamie•470
    @tea-scriptsPosted over 2 years ago

    Good luck completing the project! Some things to put into consideration would be:

    • You should consider making use of a level-one heading h1 in place of your h2 else you might have an accessibility issue.
    • Ensure to place your entire markup in a landmark like the main.
    • You should also consider adding a min-height on your columns especially for small screens. Good effort using SASS. You can try researching how to turn off source mapping when watching sass to avoid unnecessary files. This would come in handy when you have multiple .sass files that aren't modules. Great work, happy coding 👍!
    Marked as helpful
    0
    Muhamad Rizky•140
    @bodashideungPosted over 2 years ago

    @coder-teamie Thanks mate for your feedback. I'm glad that you gave me some great advices.

    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