Design comparison
Solution retrospective
Still not happy with the final result. Some feedback to improve will be appreciated
Community feedback
- @MordenWebDevPosted almost 3 years ago
nice job. you can see that the 3day left is taking new line. you can fix this by removing ul in the sub2 and making it as display flex. this code will help you to align items.
ie) in html
<div class="sub2"> <div class=""><img src="images/icon-ethereum.svg" alt="">0.041 ETH</div> <div class=""><img src="images/icon-clock.svg" alt="">3 days left </div> </div> css style:.sub2{ display: flex; justify-content: space-between; align-items: center;
}
it will be aligned in one line and you can give some margin for the image. so the is little space between image and text.
Marked as helpful0 - @skyv26Posted almost 3 years ago
Hi! I noticed some issues listed below:
-
Hover effect on image is absent. Suggestion Add image using container along with image using
absolute
andrelative
position. -
0.041 ETH 3 days left
Data is not align to center. Suggestion usedisplay
property in both li and ul element's css. Little bit work will solve your issue. -
To much shadow at left bottom side. Suggestion
use
box-shadow: 0 10px 20px 1px black;
instead ofbox-shadow: 10px 15px 5px black;
I hope you understand and my suggestions may help to solve your issues.
Good Luck
Marked as helpful0@JorahhhPosted almost 3 years agoThanks a lot for the feedback!
- I'm trying to learn how to put that hover effect!
- That's why! 3)I'll try with this
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