Hey @kkzio 👋🏻
I have some suggestions on how to fix the accessibility issues.
- Most of the issues are caused because in your
<ul>
you're not using<li>
, so if you put those<div>
s inside an<li>
, it should fix most of the accessibility issues. - I think instead of using
<header>
it should be<main>
, because that's the main part of the page. And<div class="attribution">...</div>
it should be<footer class="attribution">...</footer>
- I'd suggest putting the
logo
in a<span>
or some different tag other than a heading. You can use<h1>
on the headings of the slides.
I hope this was helpful 👨🏻💻 after you fix the accessibility and HTML, make sure to generate a new report. Since I haven't done the challenge, I cannot give feedback on the layout or styles, but it looks good to me 🤔 Cheers 👾
Marked as helpful
1
AJI SULISTIAN• 150
@kkzio
Posted
@kens-visuals Thank for your advice sir
0