Design comparison
SolutionDesign
Community feedback
- @katrine9176Posted over 1 year ago
Hello your solution looks really good! 😃
Here is some advice:
- don't put CSS styles in
index.html
, instead of that createstyle.css
and there put your styles, - to connect
style.css
with HTML file, add in<head>
-<link rel="stylesheet" href="style.css">
, - add a CSS reset, learn about that here -> article, to do that add:
* { margin: 0; padding: 0; box-sizing: border-box; }
- every page has to have one
<h1>
tag as a headline, - to center a card, try:
body { display: flex; min-height: 100vh; align-items: center; justify-content: center; }
- I recommend you to use semantic tags, learn about it here -> semantic tags
Hope it was helpful and happy coding! 😁
Marked as helpful0 - don't put CSS styles in
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