Ping single column coming soon (React + TailwindCSS + Dark/Light mode)

Community feedback

• 

  T

  Grace•31,270

  @grace-snowPosted almost 2 years ago

  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,140

  @MelvinAguilarPosted almost 2 years ago

  @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
• 

  LilithNixx•190

  @LilithNixxPosted almost 2 years ago

  This is really impresive to me, you don't have a CSS file, and the HTML is so small... all the rest I don't understand at all. :D

  0