
Responsive site achieved using Flex box layouts & MediaQueries
Design comparison
Community feedback
- @cia2003Posted 7 days ago
Hello!
I see that you want to put a creativity in your project, like using different color for the background. It is great!
I suggest you read about BEM methodology to improve your code's clarity. It will be helpful for other developers to read your code. For example, I see in your html's code like this:
<div class="price"> <span class="pic1"> $149.99 </span> <span class="pic2"> $169.99 </span> </div>
As you can see in the div.price there are two spans with class
pic1
andpic2
. It is unclear if you said it as a price or picture.For the product category (div.DIV1), you can use properties like
text-transform' to uppercase the text or
letter-spacing` to give space between the letters in CSS.Hope that helps!
0
Please log in to post a comment
Log in with GitHubJoin 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