Alternative Huddle landing page. Using flexbox, grid and CSS position
Design comparison
Solution retrospective
Feedbacks are welcome !!!
Community feedback
- @Jeen-PreshPosted almost 2 years ago
Hello Hassia, congratulations on completing the fem challenge.
Some tips to help improve your project
- best naming convention for classes and id's, is to separate the names using
"-"
instead of"_"
more info naming conventions
- you should use the
address
element to wrap the address section instead of article
more info address element MDN link
- it's not best practice to use an
article
element inside a footer. wrap them with divs instead
Happy coding. if you found this helpful, make sure to mark as helpful thanks.
4@MelvinAguilarPosted almost 2 years ago@Jeen-Presh, @Hassiai Hi!, There is a well-known and highly recommended convention for the specificity of styles and it is called BEM. BEM stands for Block, Element, Modifier. It is a methodology for writing CSS classes that makes it easier to maintain and understand the code. BEM is based on the idea of breaking down a UI component into its constituent parts (blocks, elements, and modifiers) and then applying styles to each part.
Example:
<footer class="footer"> <ul class="footer__list"> <li class="footer__item"> <a href="#" class="footer__link">About Us</a> </li> <li class="footer__item"> <a href="#" class="footer__link--active">Contact Us</a> </li> </ul> </footer>
Your other suggestions are excellent
3@Jeen-PreshPosted almost 2 years ago@MelvinAguilar wow new features every day. Thanks
0 - best naming convention for classes and id's, is to separate the names using
- @MohmedElshaarawyPosted almost 2 years ago
Hello Hassia, congratulations on completing the fem challenge.
Some tips to help improve your project
best naming convention for classes and id's, is to separate the names using "-" instead of "_" more info naming conventions
you should use the address element to wrap the address section instead of article more info address element MDN link
it's not best practice to use an article element inside a footer. wrap them with divs instead Happy coding. if you found this helpful, make sure to mark as helpful thanks.
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