@brasspetals
Posted
Hi, Lasha! Congrats on submitting your first solution! 🎉
When it comes to your HTML, you did a very good job overall. Just a few suggestions:
- Adding a visually hidden heading to your
.section-img
to describe to screen readers what that section is - something like “Used by the the following companies”. Here’s an article that describes different methods to add text just for screen readers. Adding alt text descriptions of each logo would be a good idea in this section too, so that screen readers will know what companies are using the product. - The
.footer-nav
can be anav
element instead of adiv
to give it more semantic meaning.
For responsiveness, your paragraphs get quite stretched out before switching to the desktop layout. You may want to consider adding max-widths to them. Using ch
units for this might be handy.
Marked as helpful