Great example for Flexbox. If you notice any problem, please give feedback about that.
@rmartin93
Posted
Only issue I really see is the button doesn't have a cursor hover effect on it. I would also look into using transition to control the speed at which the color changes. It's a small improvement, but one you'll use in everything once you know about it.
Happy coding!
@alwaysJunior
Posted
@rmartin93 Hi,
I actually add the hover for the button but yeah you are right I forgot to use transition, thank you for your feedback.
Happy coding my friend.
@Thewatcher13
Posted
@alwaysJunior
Posted
@Thewatcher13 Hello,
There is no spesific reason for the h3 tag, I just used that h3 tag for "Your Result and Summary". (.title class)
For the b tag, actually I never use b tag but this time I thought if I use that tag maybe I won't need any css for the font-weight, I just tried but after that I forgot the revoke it.
At the beginning, I used p both for 76 and 100 but then I couldn't set the color of the 76, so I didn't want unnecessary CSS that is why I changed it as h1. That was easier to write.
Also, I noted your landmark and icon advices. Thank you for the feedback, I hope I was able to explain myself.
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
