Design comparison
Community feedback
- @AdrianoEscarabotePosted 28 days ago
Hi Jay Khatri, how are you doing? I really loved the outcome of your project, but I have a few suggestions that I think might be helpful:
<h1>Reliable, efficient delivery</h1> <h1>Powered by Technology</h1>
The most appropriate in this case would be just an h1 tag! containing the two contents, to make them break a line, we can use a max-width, and for the styling we can use a span element with the content that will be changed!
The main tag must be present in every HTML document so that we can recognize the main content. To fix this, wrap the main content in the main tag. Users of assistive technology will have a better navigation experience on your site thanks to the use of HTML5 landmark elements.
The rest is excellent.
I hope you find it useful. 👍
Marked as helpful1 - @arnoldpam11Posted 29 days ago
Congratulations on finishing this project! You did a great job. Things I could say for your improvement is that strive to write clean codes and make it more readable to devs. Overall you did awesome!
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