Sarah Delluta• 10
@sarahrchl
Submitted
Mustapha
@muben88
All comments
- Mustapha• 540
@muben88
Posted
Hi @sarahrchl, your solution looks perfect!
Just a small note, you can fix accessibility issues by wrapping everything inside a main tag instead of using div, as it will be more semantic. Also, try to use an h1 instead of an h2 tag, it's important to go from h1 ==> h5 as much as you can. And one last notice, it would be better if you used an alt for your image, as it's important for users who depend on screen readers. Good luck!
0 - Tetiana His• 140
@tania0808
Submitted
Mustapha• 540@muben88
Posted
Hi @tania0808, congrats on this solution, it looks perfect!
Just a small notice, you can fix accessibility issues by wrapping everything inside a main tag instead of using div, as it will be semantic. Also, try to use an h1 instead of an h4 tag, it's important to go from h1 ==> h5 as much as you can. And one last notice, it would be better if you used padding for the whole container div instead of using it on each element. Good luck!
Marked as helpful
0
