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

just flex

@abbaskhan349

Desktop design screenshot for the Product preview card component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Community feedback

Lucas 👾 104,400

@correlucas

Posted

Hello Muhammad, congratulations for your new solution!

I saw your solution live site and seems good, you need to give some adjustment and then will be perfect, instead of giving each div under the container size, you can use a max-width: 900px thats is the total container sizes and the divs under the container will have half of the container each one.

About the code and semantics my tip for you, is to avoid using a lots of id prefer to use class for styling and let the id for forms and JS. Remember that less elements you use more control you've, so you can delete some unnecessary divs and you'll see how clean will be your code!

Anyway, great solution bro! Happy coding!

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