Design comparison
Solution retrospective
What I learned
- how to import fonts using @import
- how to swap images between mobile and desktop version using tag
- how to use rem and em units in css
Feedback welcome
Community feedback
- @IkuewumiPosted over 1 year ago
Hi ๐. Great project. But apart from the aforementioned tips, I noticed:
- For the word perfume, you don't need to use spaces to space it out. The
letter-spacingproperty would probably be a better fit here like so:selector {letter-spacing: 5px;} - Using the unicode entity
nbspfor spacing is a valid technique but is pretty much deprecated at this point. You could consider just removing it and letting the browser handle that by itself. But if you want more control on word wrapping, check out theword-wrapcss property
Hope you find these helpful. Feel free to ask any questions, and keep coding๐,
Ayobami
Marked as helpful1 - For the word perfume, you don't need to use spaces to space it out. The
- @Daniel-BilodidPosted over 1 year ago
Hi, my congratulations you did a great job ๐
๐๐จ๐ฆ๐ ๐ญ๐ข๐ฉ๐ฌ ๐๐จ๐ซ ๐ฒ๐จ๐ฎ๐ซ ๐๐จ๐๐ ๐
๐น You need to put a landmark in your code
simply wrap your code with <main class='product-previev-container'> landmark instead of <section class='container'>
๐ The <main> HTML element is intended for the main content (content) of the <body> of the document (page).
I hope it was helpful, you are great, keep up the good work ๐
Marked as helpful1
Please log in to post a comment
Log in with GitHub
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