Skip to content
OverviewSolutions
6
Comments
5
gabriele2k9
@Gabry091

All comments

  • gabriele2k9•80
    @Gabry091Posted almost 3 years ago

    Hey man! I really like your solution, but i think you could improve it by decreasing the height of the main card, and adding a transition to the main image hover using *** transition: .5s all ease ***. Great job, and happy new year!

    0
  • gabriele2k9•80
    @Gabry091Posted almost 3 years ago

    Hey man! I really like your solution, you really did a great job, but i think you could improve it adding more "px" to the font size of the titles. Happy new year!

    1
  • gabriele2k9•80
    @Gabry091Posted almost 3 years ago

    alright, thank you fot the feedback and happy new year!

    0
  • gabriele2k9•80
    @Gabry091Posted almost 3 years ago

    alright, thank you for the feedback, and happy new year!

    0
  • gabriele2k9•80
    @Gabry091Posted almost 3 years ago

    hey! I really like your solution but i want to give you a suggestion: when you hover the image (the image with the eye) make the eye have 1.0 opacity and add a transition, good job btw

    Marked as helpful
    1