Design comparison
Solution retrospective
While working, I encountered a problem on mobile devices due to the button sticking when pressed. Fixed the entire line that I checked in the manual that I often use Doka-guide
@media (any-hover: hover) {
button:hover {
background-color: hsl(157, 37%, 14%);
}
}
Community feedback
- @StroudyPosted about 2 months ago
Hello again, Incredible work on this! You’re making great strides, and I have a couple of suggestions that might push it even further…
- These
<span>
should really have semantic tags like headings (<h1> to <h6>
) and paragraphs (<p>
) convey structure and meaning to content, improving accessibility, SEO, and readability by helping search engines and screen readers interpret the content.
<span class="item-class">Perfume</span>
-
Line height is usually unitless to scale proportionally with the font size, keeping text readable across different devices. Best practice is to use a unitless value like
1.5
for flexibility. Avoid using fixed units likepx
or%
, as they don't adapt well to changes in font size or layout. -
Using
rem
orem
units in@media
queries is better thanpx
because they are relative units that adapt to user settings, like their preferred font size. This makes your design more responsive and accessible, ensuring it looks good on different devices and respects user preferences. -
I think you can benefit from using a naming convention like BEM (Block, Element, Modifier) is beneficial because it makes your CSS more organized, readable, and easier to maintain. BEM helps you clearly understand the purpose of each class, avoid naming conflicts, and create reusable components, leading to a more scalable codebase. For more details BEM,
You’re doing so well, and I hope this feedback is helpful! Keep honing your skills and remember to enjoy the process—you’re on a great path. Stay motivated, and happy coding! 🎯
Marked as helpful1@WaRaGiSmYnIcKnAmEPosted about 2 months agoHello, @Stroudy! I started frontend just a couple of months ago, when I had a desire to create my own website. Thanks to your comments, I began to make the layout more competently, and most importantly, quickly!
1@StroudyPosted about 2 months agoHey @WaRaGiSmYnIcKnAmE, This feedback does not mean you have done anything incorrectly, Take it slow, Its not a race, Your smashing it and when learning there is so much to take in. You did a great job with this solution, Learning the fundamentals of coding is a marathon, You will only get better, Look at where you are now and where you was just a few months ago, You got this bro! 💪
1 - These
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