Design comparison
Solution retrospective
it is much easier to vertically align content with Flexbox in a single dimension layout
What challenges did you encounter, and how did you overcome them?using CSS Grid to specify space between content is much easier
What specific areas of your project would you like help with?styling with CSS, especially with sizing units when to use px, em, rem, percent, etc...
Community feedback
- @danielmrz-devPosted 6 months ago
Hello there!
Congrats on completing the challenge! โ
Your solution looks excellent!
I have just one suggestion:
- Use only
<main>
to wrap the main content instead of<main>
and<article>
.
๐ The tag
<article>
would make more sense if the card was part of a bigger website (in certainly would in real world), but here it is all we have on the screen.Unless you have a specific reason to use both, there's no need, considering that you can achieve the same result with only one. It also makes it easier positioning the elements inside.
This tag change does not impact your project visually and makes your HTML code more semantic, improving SEO optimization as well as the accessibility of your project.
I hope it helps!
Other than that, great job!
Marked as helpful1@dwangsstPosted 6 months ago@danielmrz-dev thanks for your supportive feedback, I've adjusted my solution based on that.
Now, my solution is clean from the accessibility report and HTML report
1 - Use only
- @kodan96Posted 6 months ago
hi there ๐
You should avoid hard-coded values (pixels) most of the time. When you use these values you give up responsibility (or you make it harder for yourself at least)..
Typically you will increase the
font-size
property with@media
queries. If you have hard-coded values all over your CSS, you need to modify every element'sfont-size
. On the other hand, if you userem
-s all you need to do is changing thefont-size
in your CSS:root
selector and all your elements will have a new size based on that value.I usually use
em
forpadding
andmargin
for text-based elements, since their margin usually based on theirfont-size
, and again, when you change thefont-size
in:root
these values will scale up as well without you touching them, making your job easier and your page maintainable.Hope this helps! ๐
Good luck and happy coding!
Marked as helpful1@dwangsstPosted 6 months ago@kodan96 thanks for your supportive feedback, I've adjusted my solution based on that.
1
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