@fernandolapaz
Posted
Hi π, just a few tips in case you want to take a look:
- The icons are decorative images and therefore need an empty
alt
attribute to be ignored by a screen reader.
- It is better to use
min-height: 100vh;
for the body, as usingheight
causes the page to be cut off in viewports with small height (such as mobile landscape orientation).
- And keep in mind that it would be good if each page had an
<h1>
to improve user experience and because it is an important element when it comes to SEO. Could be a good idea to add an <h1> at the beginning of the page and hide it with CSS for example...
I hope itβs useful π
Regards,
Marked as helpful
@MikeBeloborodov
Posted
@fernandolapaz Thanks a lot! Good advice about h1 and hiding it, I saw that technique somewhere already.
In regards to height, I check landscape phone and it's cut indeed, thanks for the tip, appreciate it a lot!
Also didn't know about icons and decorative images, will remember in the future.