Design comparison
Solution retrospective
learned and applied BME
Community feedback
- @AdrianoEscarabotePosted about 1 month ago
Hi Maobugichi, hope you're doing well! I loved how your project turned out, but I’ve got a few suggestions that could be useful:
Note that in higher resolutions the card is increasing a lot, damaging the design of the project, so to fix it we can do this:
.main__item { max-width: 340px; }
Always prefer to use
max-width
, it will define a maximum width for the card, so in higher resolutions, that is, on larger screens, the card will retain the same appearanceThe rest is fantastic.
Hopefully, you'll find it helpful. 👍
Marked as helpful0@MaobugichiPosted about 1 month agothanks alot, addressing that right now@AdrianoEscarabote
1 - @abhi-zeroPosted about 1 month ago
Hey bro, you used
position: absolute
for your overlay. If you want the image hover effect to match the given task/project, you need to use theopacity
property. Set the overlay'sopacity
to 0 initially, and then change it to 1 in the hover rule. The value 0 will hide theoverlay
, and the value 1 will make it visible.0@MaobugichiPosted about 1 month agohm, same solution. did you look through the site or just the code?, i just added a bit of animation@abhi-zero
0
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