
Solution retrospective
Iam proud to complete yet another challenge.
What challenges did you encounter, and how did you overcome them?I had a problem align two vertical cards perfectly in the center but hopefully i solved my problem
What specific areas of your project would you like help with?i dont need any help so far.Thanks
Please log in to post a comment
Log in with GitHubCommunity feedback
- @AdrianoEscarabote
Hi litddomm, hope you're doing well! I loved how your project turned out, but I’ve got a few suggestions that could be useful:
<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 fantastic.
Hopefully, you'll find it helpful. 👍
Marked as helpful - @BeeAlmighty
Hi lotddomm, Great work here! I love how you had your customized CSS styling before you began working on the project. However, here are some that i noticed that you could have done better:
- You could have used semantic HTML like main, footer, header, article, section instead of divs, it helps with web accessibilities and screen readers as well.
- Your layout definitely fit into all screen sizes from 320px to larger screens but i don't think your solution is so close to the design but hey! you might decide to freestyle, who knows 🤷🏾♂️.
Join 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