Calculator app

Community feedback

• 

  Ben Andrew Merritt•345

  @b-a-merrittPosted almost 3 years ago

  One small thing: clicking on the button to change the theme doesn't work. I had to dive into dev tools to figure out that I had to click on the numbers. This was backwards for me in terms of UI / UX.

  If you agree, moving your buttons to where switch-button div lives and making their ::before contents equal to the numbers (essentially swapping containers) would solve this.

  If not, at least at a hover state to the numbers so people like me might figure it out more intuitively.

  Either way, I'm a huge fan of how clean your JavaScript is written and commented!

  Marked as helpful

  0

  ---

  

  Lourdes González•150

  @Lourdes84Posted almost 3 years ago

  Hi @b-a-merritt ! I have corrected the theme selector just as you recommended me. Thank you for your advice, it helps me a lot to improve my code. Thanks! Have a nice day!

  1
• 

  Adekola Thanni•590

  @AdekolaThanniPosted almost 3 years ago

  You've done a great job!, to even make it better try to enable the users to see what operations they've chosen... That should improve the user experience... Have a great time coding!

  Marked as helpful

  0

  ---

  

  Lourdes González•150

  @Lourdes84Posted almost 3 years ago

  Thank you for your recommendation @AdekolaThanni !

  0