Design comparison
Solution retrospective
Another day, another project. As always, another interesting and fun challenge solved. Tools: React, React-Router, Tailwind, TypeScript. React was, without a doubt, a solid tool for solving this challenge, and it allowed me to delve even further into mastering this tool, the same goes for TypeScript. It was my first time using Tailwind and, honestly, it was a bit confusing at the beginning, but then the work with this framework became increasingly organic and fluid. I tried to optimize the web application as much as possible, in addition to ensuring that navigation was as smooth as possible, but I'm sure it can still be improved, and that there will be errors to correct and pieces of code to refine. As always, I appreciate your feedback and advice in advance.
Community feedback
- @elder2701Posted 10 months ago
Hi. This project is really good; you've done a great job. However, I have some comments for you that may help improve this solution.
When using TailwindCSS, it's better to create a small design system. This allows you to reuse Tailwind classes and avoid repetition.
For example. You have next block, which repeats at least 3 times
<p className="font-semibold text-sm">
Consider creating small components for elements like buttons, inputs, h1...hN, and so on, to facilitate reuse in the future.
One additional point I'd like to highlight is the quality of the code base. Here are some general recommendations for the final version of the solution: try to eliminate unnecessary parts of the code and comments.
I hope this helps you. Happy coding!
Marked as helpful0@Dantalian5Posted 10 months ago@elder2701 Thanks for the advice! It's really helpful. I'll aim to make this project better and keep your suggestions in mind for future work. 👍
0
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