Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

3-Column Card Component Solution

@j-fraza

Desktop design screenshot for the 3-column preview card component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


All types of feedback are welcome, as I am still learning throughout my journey to become a Frontend developer! Thanks in advanced :)

Community feedback

P
Grace 27,650

@grace-snow

Posted

I think you've done a good job here, well done.

Just a few improvements needed :

  • font size can never be in px. Always a relative unit
  • add some padding to the body so content can't hit the side of the screens
  • what you have as inputs should be anchor tags. These would be navigation links, not buttons.

When you do need to use buttons in future challenges (for actions like form submissions or showing/hiding an element) don't use inputs for it then either. Use the button element. Input type button is an old legacy thing that doesn't need to be used any more.

Well done again, and good luck on the next challenge

2

@j-fraza

Posted

@grace-snow Thanks so much for your input (no pun intended 😆)! But in all seriousness, thank you, especially with the button element standards. The more you learn everyday :)

0

Please log in to post a comment

Log in with GitHub
Discord logo

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