Design comparison
Solution retrospective
Hello 👋, Any feedbacks are welcome, Thanks
Community feedback
- @vanzasetiaPosted about 2 years ago
Hi, Tenzin! 👋
Congratulations on finishing another challenge! 👏 It has been a while since my last feedback on your solution. 😄
Anyway, some recommendations from me.
- I recommend removing the
max-width: 1440px;
from thebody
element. There's no need to limit the width of thebody
element. Treat thebody
element as the page of the site. What I recommend doing is to limit the width of the elements inside thebody
element. - The alternative text for the logo should be "Huddle". This way, the screenreader users can know that they are on the Huddle website.
- I would make the register button as a link instead of a button element. As a user, I would think that after I click the button, I will get navigated to the register page. So, what do you think will happen after the users click the button? Regardless of what you decide later, always specify the
type
of thebutton
. It will prevent the button from behaving unexpectedly. - Each of the social media icons should be wrapped by an anchor tag. They have interactivity so they need to be wrapped with interactive elements.
That's it! I hope this helps! Happy coding! 😄
Marked as helpful2@vanzasetiaPosted about 2 years ago@lekdup
One more suggestion I have is to keep the specificity of the CSS as low as possible. This was the mistake that I made when I started using Sass. I nested too much and as a result, when I came back again, I wasn't able to understand the code that I wrote. So, I would recommend only nesting when you want to style pseudo-elements, pseudo-classes (
:hover
,:focus
, etc) and@media
queries. It's a good practice to keep the CSS specificity as low and flat as possible. High specificity will make your stylesheet hard to maintain.Marked as helpful0@lekdupPosted about 2 years ago@vanzasetia Hello Venza, Thank you very much for your time and helping me out! your feedbacks are always very helpful for me and always get to learn new things from you. thanks again i hope to see your feedbacks on my new challenges.
1 - I recommend removing the
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