Any feedback is welcome
Fabricio Rivera
@FabricioRivera2021All comments
- @rsrclabSubmitted almost 3 years ago@FabricioRivera2021Posted almost 3 years ago
Hey, good solution, how do you do to make the 2 backgrounds circles to stay in the center, i couldnt do that.
0 - @FabricioRivera2021Submitted almost 3 years ago
Hi i got a question about this one, and the nft proyecto too... i dont know if it was ok to put the width for the desktop device, at first i was only gonna do the widht: 100vw; aproach like i did on the last one.
what is most correct, put the width of the devices or just let it adjust with the viewport units?.
@FabricioRivera2021Posted almost 3 years agoHi Thank both of you for the feedback, i really appreciate it. I made the changes to the css file to see if it fix the issues with the touching of the card on the mobile version. And change the background, the property max-width was really helpfull thanks. I think is better now, its not a 100% match though.
0 - @ItsIchiSubmitted almost 3 years ago
Two challenges I ran into was first the HR I couldn't figure out how to get it anymore thin, and the second was the active state for the image I ended up just creating an overlaying div that acted as the hover state but if there's a better or more simple way to do so let me know!
@FabricioRivera2021Posted almost 3 years agoI ran into trougles with the hover image too, dont know if its a better way but i lower the opacity of the image in the hover, and put cyan on the container of the image and show the eye with a background image.
0