Nice solution, looks good
I’m not convinced that theme toggle is accessible, as there is no communication of what is active or what the label might be for voice users. I’d need to test that to know for sure
With the form it’s great. Only issues are there’s no label I can see on that input, and no persistent div with aria-live on it wrapping the errors
Marked as helpful
2
Melvin Aguilar 🧑🏻💻• 61,440
@MelvinAguilar
Posted
@grace-snow Hello!. My apologies for the delay. I really appreciate the review of my solution. I just updated the code with your suggestions. I ended up changing the theme by using an input tag, If you have any other suggestions, please let me know and I'll be happy to make the necessary changes. 😊
1
