Design comparison
SolutionDesign
Solution retrospective
I am a student and I try to keep improving, any suggestions are welcome.
Community feedback
- @SoleguiaPosted over 1 year ago
Hi!
You're doing it nice but if you allow me I want to give some tips:
- You should use only classes for the css, avoiding ids and keeping them for the js. That way you keep the specificity a little bit easier in case of future needs of changes on the css.
- For the "PERFUME" word styling you could use letter-spacing: [...]px/rem/... ; and text-transform: uppercase;
- And for the responsive you could hide/show only the image and change the flex container direction instead of duplicate all the content.
Regards ;)
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