@mattstuddert
Posted
Hey Chamu, looking at your code I've got a few suggestions:
- Always be sure to use an
alt
attribute whenever you add animg
tag in your HTML. Otherwise, screen readers will read out the file page in thesrc
attribute, which isn't helpful for people using screen readers. - While I'm talking about screen readers I'd also recommend not uppercasing your words in your HTML, as you have done with the "Newsletter" heading. If you write the word as "NEWSLETTER" screen readers would read the text one letter at a time instead of reading out the word. Instead, you can use
text-transform: uppercase
in your CSS. - At the moment, you're using multiple
h1
elements on the page. I would recommend keeping to only having oneh1
on any page. - I'd also recommend staying away from using IDs for styling purposes as they are over specific and cannot be reused on the page.
I hope these pointers help. Keep up the great work!