Skip to content
OverviewSolutions
32
Comments
2
Mustapha
@muben88

All comments

  • Mustapha•540
    @muben88Posted almost 3 years ago

    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
  • Mustapha•540
    @muben88Posted almost 3 years ago

    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