Design comparison
SolutionDesign
Solution retrospective
All feedbacks are welcome
Community feedback
- @MarcoDV47Posted 12 months ago
Hey, it looks nice! Some tips i can help you with:
HTML
- span heading: two heading tags would be better here, they give more meaning and accessibility :
<span class="head-1"> Reliable,efficient delivery </span> <span class="head-2">Powered by technology </span>
CSS
- header: instead of using flex to center text you could use text-align on the
header
tag. - <img> alt tags: as they're decorative you can leave the alt="" blank.
Other than that it's a really great job! I liked your decision to go with flex. Maybe try with Grid some other time and compare to see what you feel more comfortable with.
Marked as helpful0
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