Design comparison
SolutionDesign
Community feedback
- @katrine9176Posted over 1 year ago
Hello! Your solution looks really good! 😁
Here is some advice:
- don't put your CSS styles in HTML file, instead of that create
style.css
file and there put them, - to connect
style.css
with HTML add in<head>
-<link rel="stylesheet" href="css/style.css">
, - I recommend you to use semantic tags, learn about that here -> article,
- an article about IDs and CSS classes -> article,
- in CSS reset you don't have to put
%
:
* { margin: 0; padding: 0; box-sizing: border-box; }
Learn about that more here -> article.
Hope it was helpful and happy coding! 😀
Marked as helpful1 - don't put your CSS styles in HTML file, instead of that create
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