Design comparison
Community feedback
- @jordanj03Posted 7 months ago
Very nice project however, some improvements I have:
Font-size: maybe try use the clamp function next time for your font size to make it more responsive HTML Markup: I would avoid writing "P E R F U M E" and instead using CSS properties to achieve the same effect (i.e. letter-spacing, text-transform) Also, for the image of the perfume, I would avoid using a div and floating the image as this does not account for people with screen readers. Ideally, you should use the <picture> tag and include a srcset to change the image depending on screen size
Overall, nice project, just some things I would have done differently :)
Marked as helpful0
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