
Please log in to post a comment
Log in with GitHubCommunity feedback
- P@inappdesign
Hi Stax,
I like your use of internal CSS styling. I prefer to keep my CSS in a seprate file but for a small project like this it isn't necessary and your HTML is very clean.
As optional changes, you could consider changing the container "div" to "main" to make it semantic, and the use of relative units like em or rem instead of pixels when sizing.
Marked as helpful
Join 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