Design comparison
Solution retrospective
Hello everyone! 👋
I finished the Results summary component! It felt amazing to complete another challenge this month <3.
- What did you find difficult while building the project?
The responsiveness of the website took me too long to process, but I think I nailed it 😅. Also the "icon-view" it's hard to place over the "nft-image".
- Which areas of your code are you unsure of?
The active status on my "icon-view" when you hover you see the "icon" but the background color is not applying, If you have any clue how can I make this possible please feel free to put any inputs!!
- Do you have any questions about best practices?
Please feel free to input any best practices that you find not best on my code I will really appreciate any feedbacks and best tips from you guys!!.
Happy weekend coding, guys!! 👨💻💙
Community feedback
- @0xabdulkhaliqPosted over 1 year ago
Hello there 👋. Congratulations on successfully completing the challenge! 🎉
- I have other recommendations regarding your code that I believe will be of great interest to you.
iMAGE 📸:
- "The active status on my "icon-view" when you hover you see the "icon" but the background color is not applying", After reviewing your code i found a way to apply the hover along with
background-color
- You can apply the following css rules to get desired output.
.nft-image { background: url(/assets/icon-view-fffad88f.svg), #00ffff7d; background-repeat: repeat, repeat; background-position: center, center; background-repeat: no-repeat, no-repeat; .nft-image img:hover { opacity: .38; cursor: pointer;
.
I hope you find this helpful 😄 Above all, the solution you submitted is great !
Happy coding!
Marked as helpful1@iEarlGPosted over 1 year ago@0xAbdulKhalid Thanks, man I will definitely try this later. Thank you again man. And thank you for the compliment man.
0 - @kabir-afkPosted over 1 year ago
Heyy!! First of all , great code man , after reading it I think if you would have added
background-color:cyan;
in your .nft-image class then it might've helped, afterall it is a div tag , if you were able to set background img then background-color would have also been possible- HTML
<div class="image"> <img src="images/image-equilibrium.jpg" alt="pic of cube"> <div class="icon-view"><img src="images/icon-view.svg" alt="icon-view"></div> </div>
- CSS
.image { position: relative; } .icon-view { position: absolute; top: 0; left: 0; opacity: 0; background-color: cyan; border-radius: 10px; width: 100%; height: 99%; box-sizing: border-box; padding: 44% 45%; } .icon-view:hover { cursor: pointer; opacity: 0.375; }
Hope this helps!
Marked as helpful0@iEarlGPosted over 1 year ago@kabir-afk Hey man, thank you for the compliment. I will also definitely try this code later. Thank you again man.
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