Design comparison
Community feedback
- @nenadmnePosted almost 2 years ago
Hello, i had some spare time so i look into your code and shere few thoughts.
h1 { font-weight: bold; }
// I would use just
font-weight: 700;
as you did in.category
further bellow. //@media only screen and (max-width: 700px) { .prices { display: flex; align-items: center; }
Here i saw you repeated some of the atributtes from previus code. When using @media you should write down only code that is changing further.
Last thing i would suggest is that instead writting for example
display:flex
, for every class, you just tight them all under same line:.one, .two, .three, .four { display:flex; }
nice project and happy coding =)
1
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