Skip to content
Submitted over 3 years ago

Calculator App using HTML, CSS, JS

Jon•60
@Jon-Ledo
A solution to the Calculator app challenge
Preview (opens in new tab)Code (opens in new tab)

Solution retrospective

There are definitely some things I could have set up better in my solution. I'm still quite proud of completing the challenge solo after a long hiatus without coding. Hope to update this soon, and sort the JS into modules

Code

Please log in to post a comment

Log in

Community feedback

  • Adrian•390
    @adriandotdev

    Hey Jon! Really well done!

    I notice some issues:

    First, so I test your calculator app, so the user can't type in the keyboard?

    Second, I notice you also code in JavaScript for the hover effect, I think you can also do that in CSS. So, if the user changes the theme, you can use the classList attribute to toggle the classes for a certain theme.

    Third, for the theme button, I think it is better to make the numbers clickable so that the user doesn't have to click on the toggle button. It is also for accessibility purposes, since you are using a button for the toggle's circle, you can use a <label> tag to point to a certain button so that the user can click the number.

    Marked as helpful

Join our Discord community

Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!

Join our Discord