sunnyside solution in HTML and CSS

Solution retrospective
All feedback is walcome! thanks you in advance
Please log in to post a comment
Log in with GitHubCommunity feedback
- P@Alexandra2888
Hi! Congrats for finishing challenge! Your solution is very close to the design! Here are some things i think you can improve:
- implement semantic HTML,
- make CSS reset: *, *::after, *::before {margin:0; padding:0: box-sizing: border-box;};
- try to use either classes or ids,
- choose better names for classes or implement BEM naming convention. It will be easier in bigger projects and when you will work with other developers,
- convert px into relative units (em/rem). Px are usually used when coding for magazines/newspapers and you need fix units. Happy coding!
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