Dimitar Radev• 930
@Mitko90
Posted
Hi, nice solution.
It looks great.
Maybe you can check some suggestions I have for you:
- You used three times the
<h1>
heading. This heading should only be used once. Maybe change it to<h2>
as it can be reused. Of course every page should have anh1
heading so I suggest adding avisually-hidden
one to the page, so it doesn't generate an accessibility report. - Your buttons should direct the user to a different part or page of the site. So maybe switch the
<button>
with the<a>
tag. - You generated a report All page content should be contained by landmarks
. To fix it simply replace your
<div class="container">
with<main class="container">
.
I had similar mistakes when I did this challenge so I redid it with the changes. You can, if you want, check both of my submissions here and here.
Above all the solution you submitted looks great.
Happy coding.
Marked as helpful
1
Barry Thatcher• 70
@Bazza-08
Posted
@Mitko90 Thank-you! Appreciate the feedback!
1