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

Responsive Preview Card Page

P
Jay 260

@Hyuuga81

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


Is there any way to prevent div height from changing when using media query and making the window larger (from mobile to tablet to desktop)?

Community feedback

PhoenixDev22 16,950

@PhoenixDev22

Posted

Hello Hyuuga81, Congratulations on completing your Frontend Mentor challenge Nice work I have noticed few things:

  • Add border-radius to the <div class=“container”>

  • Replace the button tag by an anchor tag (as they seem to navigate the user to learn more about the product). And buttons are best for forms.

  • Wrap everything in your body in <main> ... OR use semantic tags ... OR give role="" to the direct children of your <body> Read more

I really hope this feedback helps.

Marked as helpful

0

P
Jay 260

@Hyuuga81

Posted

@PhoenixDev22 Thank you SO MUCH for the useful feedback. I made all the changes and I was able to get the border radius on my container. Good advice about the anchor too. Appreciate you taking the time to look over my code!!

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