Responsive Blog Preview Card with hover effect
Design comparison
Solution retrospective
What I learned from the previous challenge (link post) i was able to adapt here. Next time i would set up the html part first before I start styling it. I have changed few times the html part before.
What challenges did you encounter, and how did you overcome them?I was not sure if I should code it based on the elements or based on classes. I think for the next time I will be consistent and go based on classes because of individual changes per element.
Community feedback
- @0xabdulkhaliqPosted 6 months ago
Hello there š. Congratulations on successfully completing the challenge! š
- I have a suggestion regarding your code that I believe will be of great interest to you.
ADDING HOVER STATE šŖ:
- Looks like the component hover state with
box-shadow
has not been set yet. Your solution would be perfectly finished if you add this as a finishing touch!
- Just add the following css rules
.preview__card { transition: box-shadow .3s; } .preview__card:hover { box-shadow: 12px 13px 0px 1px black; }
- Now your component's hover shadows has been set properly
.
I hope you find this helpful š Above all, the solution you submitted is great !
Happy coding!
Marked as helpful0 - @danielmrz-devPosted 6 months ago
Hello there!
Congrats on completing the challenge! ā
Your solution is really impressive!
I've got a couple of ideas (about how to use HTML better) that could make it even stronger:
š First: Think about using
<main>
to wrap your main content instead of<div>
.Imagine
<div>
and<span>
in HTML as basic containers. They're good for holding stuff, but they don't tell us much about what's inside or its purpose on the webpage.š Second: Don't skip heading levels ā start with
<h1>
, then use<h2>
, and so on.It's more than just text size ā it's about structuring your content effectively:
- The
<h1>
to<h6>
tags are used to define HTML headings. <h1>
is for the most important heading.<h6>
is for the least important heading.- Stick to just one
<h1>
per page ā it should be the main title for the whole page.
These tweaks might not change how your page looks, but they'll make your HTML code clearer and help with SEO and accessibility.
Hope that's helpful!
Keep up the great work!
Marked as helpful0 - The
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