Design comparison
Solution retrospective
Used React for this challenge. Any feedback for improvement is appreciated.
Community feedback
- @hardy333Posted about 3 years ago
Hey, super nice solution, everything is on point.
- Layout is responsive on all media screen sizes.
- Design looks very nice and same as in design files.
- Javascript side App logic works good.
One thing I will suggest is that you might try to make attribution font size little smaller, but it is only my personal preference and nothing more, this way as it is now looks also nice.
Dotted outline on hover effect also looks nice and interesting, although I am not sure if it is a little to much for user experience or not.
And one thing I want to ask, I see you imported bootstrap on your scss file but seems like you didn't use any bootstrap classes, so you just use bootstrap as a default/base/reset css ?
3@santu369Posted about 3 years agoHi @hardy333
Thanks for the feedback. Will surely look into the suggestions in upcoming projects. I usually use bootstrap as a base in my projects for responsive font sizes and the reboot it provides. I forgot that I was even using bootstrap 🤦♂️ else I would've used containers/classes it provides.
1
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