Design comparison
Solution retrospective
feedbacks are welcomed
Community feedback
- @Bishalsnghd07Posted 9 months ago
Hi, @hgcfkuyfliyfl
Congrats for complete this challenge🎉
A few suggestions for you might be helpful.
- To make your HTML more semantic always follow heading level, instead of using
h3
directly useh1
tag. Each pageh1
tag should be mandatory to maintain semantic ui. And Heading level should be like this<h1> <h2> <h3> <h4> <h5> and <h6>
and you should not be skipped any of this label. Always start withh1
and end withh2
.h1
is the most important tag it is always use for main title and heading andh6
would be least important.
2.Second, you used
br
tag many time to separate the paragraph text and making it into different line, instead of that you can use much better approach by giving max-width so that your paragraph text will wrap in different line and work as same as br tag. It will improve your code quality.Hope my suggestion will help you!
0@hgcfkuyfliyflPosted 9 months ago@Bishalsnghd07 "Thank you for the suggestions! I'll definitely keep them in mind to enhance the semantic structure and improve the code quality of my project."
0 - To make your HTML more semantic always follow heading level, instead of using
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