Design comparison
Solution retrospective
💻 Hello, Frontend Developer Community,
This is my solution for the Calculator app.
-
Scored 97,5% on Google Pagespeed Insights! 🚀
-
Solution with 100% W3C validation accuracy 🌟
-
Custom-built CSS animations for a smooth user experience 💫
-
Three custom schemes to choose 🖼
-
You can use calculator with your keyboard ⌨
-
Fully responsive design crafted with a mobile-first approach 📲
-
Enjoyed every moment coding this! 😎
-
Feedback is always welcome—let’s grow together! 🌱
-
Completed 5 out of 13 Intermediate Challenges so far—keeping up the momentum! 🔥
-
👨💻 Join me on my coding journey as I tackle advanced challenges and add innovative touches to every project.
-
As I am starting my journey with React I'm really looking forward to hearing your thoughts and suggestions! 💡
Community feedback
- @TedJenklerPosted 2 months ago
Hi @MikDra1,
Nice project! I’m planning to do this one myself soon. I noticed that you can use a dot (.) before any number is entered.
As an extra improvement, I’d recommend adding a useRef to make the buttons controllable with arrow keys. This would make the project more accessible and even more impressive.
Hope this was helpful!
Best, Teodor
Marked as helpful1@MikDra1Posted 2 months ago@TedJenkler
Dot is perfectly fine. If you write dot as a first value the JS will assume that you want zero before it ☺️
As you can see in the code I’m not using React, but I am planning to do this project soon in React. Maybe then I will use your advice.
1@TedJenklerPosted 2 months ago@MikDra1: My bad—I only checked the responsiveness and functionality. It's still possible to do with key event listeners and DOM manipulation, but it’s much harder.
Have a nice day / Teodor
0 - @YuliaLantzbergPosted 2 months ago
Hi. It's looking and working great. Just maybe it's better to make a guard preventing division on 0. Also, the dot is not working on the keyboard. About the js, I'd add one class to the "calculator__grid" div and in CSS would target the child divs through it. Then in js it'd take less code lines to add and remove all the classes. But all that are minors. Your work is really good as always. I'm learning a lot from your challenges. I hope I will get to this challenge soon.
Marked as helpful0
Please log in to post a comment
Log in with GitHubJoin 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