Rostyslav Nazarenko• 560
@rostyslav-nazarenko
Posted
Hi! Cool code!
I have a few suggestions.
- HTML
- use
main
element for accessibility. Change<div class="container">...</div>
to<main class="container">...</main>
alt
attribute should describe an image or its function. Don't use the word image because screen readers read the tag as "image". In this case useQR code to frontendmentor.io
. Read this article for more information- use
h1
instead ofh2
, don't skip levels
- use
- CSS
- use relative units to make your solution accessible
- set
max-width: 100%
on images
Hope I've been helpful!
0