Skip to content
OverviewSolutions
7
Comments
3
Jakub Radziejewski
@jakubinhoo

All comments

  • Jakub Radziejewski•120
    @jakubinhooPosted almost 3 years ago

    Hello, nice job. Consider adding responsiveness. Also I think that there should be a little more white space around main container, because it looks a little bit overwhelming :)

    0
  • Aneta•180
    @anetaanetteSubmitted almost 3 years ago

    Hi, this is my second project here. I would be grateful if you could let me know what I can improve in my code. Thanks

    Order Summary Card

    3
    Jakub Radziejewski•120
    @jakubinhooPosted almost 3 years ago

    I miss animations on those buttons, and also there are little box-shadows in the original design :) Overall good job, keep it up!

    Marked as helpful
    0
  • peondg•20
    @peondgSubmitted almost 3 years ago

    Everything looks right. The only thing I couldn't figure out was getting the eye or view icon to show in the middle of the image when hovering over it. If anyone can help with this, let me know.

    NFT Preview Card Component

    3
    Jakub Radziejewski•120
    @jakubinhooPosted almost 3 years ago

    Hi, good job! The way I did this was setting position on icon to absolute, center on the image and set its opacity to 0. Then I just changed its opacity to 100% in hover selector. It's a little bit tricky, I created a whole new div do achieve this, but eventually it did work out. Hope it will help.

    Marked as helpful
    1