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

Product-preview-card-component-solution

@MerMory-Alloc

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


writing concise and clear CSS is difficult because it require a good understanding of CSS attributes on which i learned in this simple exercise.

Community feedback

asekabtw 290

@asekabtw

Posted

Hello, @MerMory-Alloc! Good job. I mean really good job. I have some advices for you:

  1. You can put class container inside of the main tag.
  2. Try to use method called BEM. If you don't know what is it, you can read it here: https://getbem.com/introduction/
  3. Try to write text in p or h tags. You can change div with class category to p with class category and it will be easier, because perfume is not doing any role of some kind of link or etc.

I Hope I could help you. Even a little. I am not a professional and I am not 100% correct, so take it as a friend tip)

Marked as helpful

1

@MerMory-Alloc

Posted

@asekabtw Thanks i really do appreciate your feedback

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