Design comparison
SolutionDesign
Community feedback
- @grace-snowPosted about 2 months ago
Have a look at the feedback I've left on others solutions to this like this one as a lot of the same issues apply to yours: https://www.frontendmentor.io/solutions/html5-css-and-flexbox-aXZTaM1HQ3.
Definitely move the styles out into their own file, try to get the colours to match and you don't need overflow hidden anywhere so remove all those strange commented out lines of code. Also never use em for font size, use rem!
1 - @mshah007Posted about 2 months ago
I would use an external css file instead of using the style tags within html
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