Skip to content
Submitted over 2 years ago

3-Column Card Component

Elysson Tanaka•90
@elyssontanaka
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

Hello everyone! Any feedback or tips to help me improve the quality of my code is appreciated. I'm sure there are better ways to write cleaner and more concise and organized code.

Thanks!!

Community feedback

  • IRVINE MESA•1,855
    @DrMESAZIMPosted over 2 years ago

    hi @elyssontanaka well done . I do have one suggestion on mobile responsiveness .

    In the file styles/style.css line 136 , you used "@media (min-width: 1200px) " which I think is too large since most users have smaller screen . preferable if you had set the min-width to 600px . It would be fine

    Marked as helpful
    0
    Elysson Tanaka•90
    @elyssontanakaPosted over 2 years ago

    Hi @DrMESAZIM, it's a good point, I'm going to change the code to make it more suitable. Thanks for the tip!

    0
  • Cristian•130
    @cristiandjrPosted over 2 years ago

    Excellent work brother, the only thing that when I'm in phone mode I would leave it more to the center :D

    Marked as helpful
    0
    Elysson Tanaka•90
    @elyssontanakaPosted over 2 years ago

    Hi @cshjdr, thanks for the tip, I'll take a look at it!

    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