Design comparison
Solution retrospective
I'm proud that my css flexbox always work with this kind of project. maybe i'll try another styling next time.
What challenges did you encounter, and how did you overcome them?I encountered having trouble with the image and also the favicon. it won't show on the website. But i managed to restore it.I will double check the location of the files next time.
What specific areas of your project would you like help with?I think i used a lot of div elements.Please tell me what to do to make it better.
Community feedback
- @mkborisPosted 4 months ago
Nice work adriantech-beep, you are right It's not good practice to wrap every element in
div
because divs aren't semantic. Here's how you can make it better- All content should be wrapped within landmarks. Wrap a
main
tag around .container. - This isn't a
form
, a form would typically include elements likeinput
,textarea
, orbutton
for user input. You could use adiv
instead ofform
. - For the .navLinks use unordered list for better semantics and accessibility
ul
instead of divs and wrap eacha
inli
. - It's best practice linking Google fonts directly in the HTML
head
section rather than directly in your CSS file as it enables asynchronous downloading, improving page load times. - Media queries, font-size should be defined in
rem
not px
Hope this helps
Marked as helpful0@adriantech-beepPosted 4 months ago@mkboris Thank you so much for this. It's a great advice for me as a newbie. I will apply this tips on my next challenges and also try to redo my solution.
1 - All content should be wrapped within landmarks. Wrap a
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