Design comparison
Solution retrospective
Had fun doing this project, any suggestions are welcome.
Community feedback
- @danielmrz-devPosted 11 months ago
Hello @bhaskrr!
Your project looks great!
Just a few suggestions for improvement:
-
The
main
tag is used for main content (including text and images), not just for main text or titles. In this project, the whole card is the main content, so you can wrap the whole thing withmain
. -
header
andfooter
are tags for top and bottom content of the whole page, not header and bottom of the card. So you can place that footer outside of the card, so it won't take up space unnecessarily.
I hope it helps!
Other than those details, you did a very good job!
Marked as helpful0 -
- @accamamuhammadPosted 11 months ago
Nice solution, however I noticed a few problems with it. These are the issues and their fixes.
1.The attribute should not be in the blog preview card it should be outside at the bottom of the page: To fix this remove the attribute div and place it outside then set it
position: absolute
thenbottom: 0
andposition: relative
on the preview card container to fix this.2.Use media Queries to apply style for different screen sizes learn more about them on youtube
3.Do not separate your paragraphs use
text-wrap: wrap;
to make it change based on screen size4.USe the proper Font-weight for different texts based on designs
Marked as helpful0
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