Design comparison
Community feedback
- @EddieBones1Posted about 1 month ago
Great Start! However I have a few suggestions:
1.) In your HTML file, you don't have a anything written in your alt text. Alt text provides better SEO because it provides more context about the content, and it helps with accessibility for screen readers.
2.) Also, in your HTML file, you have a
<h2>
tag above the<h1>
which is not correct because heading tags should be used in hierarchical order from <h1> to <h6>. You can replace the <h2> tag with a <span> tag because <span> tags does not affect the heading hierarchy.<span>Reliable, efficient delivery </span> <h1> Powered by technology </h1>
Now span is an inline element. So if you want to change it to a block element just use
display:block
ordisplay:inline-block
.Hope this helps.
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