I'm afraid the theme toggle is completely inaccessible. It's fine to try and extend these challenges but not at the expense of essential accessibility. Look up how to fix it and have a go. The html needs to change the most but also the design.
Another critical accessibility issue is writing font size in px, which is an immediate WCAG failure, and especially bad when used on the html selector. By doing that you've made it impossible for people to adjust the text size. Why font-size must never be in px
Thirdly, media queries need to be defined in rem or em not px so that a site can reflow correctly no matter what text size or viewport size is used.
Marked as helpful
@Ritika-Agrawal811
Posted
Thank you so much @grace-snow for going through my code and finding these errors!
I read the articles you linked and searched some more to write an accessible toggle button. I've corrected all the issues. Thanks again!
Well done, that looks nice @Ritika-Agrawal811
I'm not sure the button label is ideal but you've sparked me into going and researching that as I'm genuinely not sure what the best button label would be
