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 css changes. Note use of rem in font size

@DavidBurgess1984

Desktop design screenshot for the Product preview card component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


Hi, thanks for looking - I'm looking for feedback on my solution. I want to know if my use of rem as the font sizing is a good approach. I have hardcoded a lot of other sizes in px, but I have read on various sites it is a good idea to use em for spacing like paddings and margins.

How is the use of HTML? I wasn't too sure about whether to use <section> elements instead of <div>.

In general, I'd like some feedback or any suggestions for improvements. I want to try and get better at styling sites in css.

Community feedback

Sonat 50

@sonatipek

Posted

Nice work! But the button doesn't have the style it should have when hovering over it. Maybe you want to fix it. Except that one a great solution. Coming to your question. " https://stackoverflow.com/questions/6939864/what-is-the-difference-between-section-and-div " Here you can find the answer to your question in detail.

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