All comments

• 

  Junior G• 40

  @Jrgwz

  Submitted over 2 years ago

  

  NFT preview card component

  1

  1

  0

  

  Adina• 50

  @adinabbb

  Posted over 2 years ago

  Hey Jrgwz! Great job, it's really close to the original design. I see you have a ease-in transition - I suggest maybe leaving only "ease" to capture the hover in and out of the element in order to look more seamless.

  Regards,

  0
• 

  hamidreza• 20

  @hamidrezatakloo

  Submitted over 2 years ago

  hi guys i will happt if u see my code and get your feedback

  

  flex box in css

  1

  1

  0

  

  Adina• 50

  @adinabbb

  Posted over 2 years ago

  Hello Hamidreza! Great work so far, see my feedback below:

  1. Try to get as close as possible font-wise - you can use "font-weight:" and choose the different types of front given in the styling file.
  2. There is no hover effect for some of the applicable elements (ex: main "Equilibrium" text and creator name).
  3. Although this is not a mistake, as a best practice tip you can make another file "style.css" where you can put all the css information there in order to keep everything organised.

  Keep up the great work and on to the next challenge :)

  Marked as helpful

  0