Design comparison
Community feedback
- @Szeri323Posted 11 months ago
Hi @VanshikaSabharwal,
Good job, I like your solution!
If you are wondering what you can improve in your solution:
you could add padding on paragraps, it will look much better.
The second thing is to center the card in the middle of the page adding
min-height:100vh
on the body and use for exampledisplay:flex
and center it withjustify-content: center
andalign-items : center
or usedisplay:grid
and center it withplace-items:center
.I hope that it will will help.
Happy New Year! ๐ฅณ
1 - @MelvinAguilarPosted 11 months ago
Hello there ๐. Good job on completing the challenge !
I have some suggestions about your code that might interest you.
- Use semantic elements such as
<main>
and<footer>
to improve accessibility and organization of your page.
-
It's recommended that you always use the font provided by the challenge's style guide.To import a font, follow the steps below:
- Go to the font's page on Google Fonts: https://fonts.google.com/specimen/Outfit.
- A sidebar will appear with a code snippet that you can use to import the font.
- Copy this code snippet and paste it into the <head> section of your HTML document.
- Now you can use the "Outfit" font in your CSS by specifying
font-family: 'Outfit', sans-serif;
.
I hope you find it useful! ๐ Above all, the solution you submitted is great!
Happy coding!
0 - Use semantic elements such as
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