Simple calculator

Community feedback

• 

  Susan Inez•0

  @sonographerPosted over 3 years ago

  UI 10/10 Logic 6/10

  I did not check your code but looks like you are using eval () function. This is not good practice to do so.

  0

  ---

  

  Japjot singh•95

  @Japjotsingh02Posted over 3 years ago

  @sonographer thanks,for your feedback. I will surely find best way to do so and can you provide me its alternative.

  1
• 

  Amon•2,560

  @A-amonPosted over 3 years ago

  Hello! It's a great work~

  But the buttons look compressed on mobile size. There are also tons of adding/removing classes in JS just for theme alone. I believe adding/removing only one class on the larger parent such as body should be enough in JS, and the rest of the theming can be done in CSS.

  0