Skip to content
OverviewSolutions
16
Comments
6
nirglus
@nirglus

All comments

  • nirglus•270
    @nirglusPosted about 1 year ago

    Congratulations mate, looks pretty good!

    I would recommend you to add a transition to the button so it will look smoother and more interactive. Also try to add the box-shadow to the container.

    Best regards.

    1
  • BryanCarlos•80
    @BryanCarlosSubmitted about 1 year ago

    The only thing I messed up it's when the nft image get the :hover status. If you hover over the view icon it keeps "shaking".

    If you know how to solve this, I would be so grateful to get your soluction.

    NFT Preview card

    2
    nirglus•270
    @nirglusPosted about 1 year ago

    Hey congratulations for completing the challenge!

    In order to fix the :hover problem, try to seperate the image divs. Make one for the image, also use background-image instead of <img> element for the Equilibrium image. Then in the image div, add the div of the eye and set a background color to it. Make the div the same width and height as the parent div, and change the opacity to 0. After that, set the opacity to 1 on hover and it should work.

    Keep up the good work :)

    Marked as helpful
    2
  • nirglus•270
    @nirglusPosted over 1 year ago

    Congratulations for completing this challenge! I have a few tips for you. Firstly, when you set a gap between items use vh for height / vw for width / px instead of rem, because em and rem are used for text size.

    Secondly, for more semantic HTML try to use the <s> tag at the outlined price.

    Thridly, at the PERFUME h4 try to use letter-spacing property to achive closer design.

    And for last, try to add some transitions and on hover properties for the button so it will be more interactive and smoother for the user.

    Keep up the good work :)

    Marked as helpful
    0
  • nirglus•270
    @nirglusPosted over 1 year ago

    Congratulations, looks good! I would recommend you to add box-shadow property to the card container.

    0
  • nirglus•270
    @nirglusPosted over 1 year ago

    Nice one ! Try to add shadow to the card component using the box-shadow element.

    Marked as helpful
    0
  • Yung Josh•20
    @yungjosh234Submitted over 1 year ago

    This project was my very first challenge as a beginner which took me about 4 hours but by God's grace I found it amusing and quite interesting and now i'm more curious to find out how the rest challenges will be like. Thank you.

    result summary

    2
    nirglus•270
    @nirglusPosted over 1 year ago

    Hey, congratulations on you first challenge! I have a few tips to achive closer results:

    Firstly, I recommend you to read about linear-gradient property. With the linear gradiate you can achive the same backgrounds of the left side and the score circle. You also have the proper colors at the style-guide file.

    Secondly, try to add transparency to the summary blocks by using rgba colors. For example: rgba(255, 87, 87, 0.06) the forth parameter sets the transparency. It goes from 0 to 1, try to play it to match the perfect color.

    Keep up the work and good luck!

    Marked as helpful
    0