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 ?
@santu369
Posted
Hi @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.
