@mattstuddert
Posted
Nice solution! You've matched your solution to the design really well. Nice work! 👍
As you mentioned, the calculator is working pretty well. But there are a few bugs in there. I'd recommend playing around with it some more and try ironing out those issues.
Also, I'd recommend reviewing the toggle. I'd expect to be able to click on 1, 2, or 3 and have it switch to the correct theme. But at the moment, it cycles between each one regardless of the option I choose. You could look into using radio inputs instead of a single button
. The button
is also causing a HTML validation error at the moment because you're nesting a div
inside it, which isn't valid HTML.
One final thing: all of your calculator buttons are currently div
elements with event listeners on them. Avoid setting click listeners on non-interactive elements, like the div
element. These can't be accessed by anyone not using a mouse/trackpad to navigate the content, which is a bad practice. Instead, add click listeners to interactive elements like a
or button
. This will ensure the element is focusable and accessible by people not using a mouse/trackpad.
I hope this helps! Keep up the great work! 🙂
@olgak169
Posted
@mattstuddert Thanks for your feedback! All great suggestions, will definitely improve my solution with your help!