NFT preview card component challenge hub

Community feedback

• 

  Alex•660

  @al3xbackPosted almost 3 years ago

  Hi rizahoemae,

  In my opinion, using the opacity property to reduce the contrast of a color is not a good idea, because there is a condition where if in future we want to add an image, icon or other element inside that element, then the newly added element will also be affected by the opacity.

  It is better if we set through tailwind text-opacity:

  from:

  <div class="text-white opacity-50 text-[14px] mt-2">Our Equilibrium collection promotes balance and calm.</div>
  

  change to:

  <div class="text-white text-[14px] mt-2 text-opacity-50">Our Equilibrium collection promotes balance and calm.</div>
  

  This approach will only inherit color.

  1

  ---

  

  Mawaddatur Riza•20

  @rizahoemaePosted almost 3 years ago

  @al3xback thanks for your suggestion, already implemented in my newest work :D

  0
• 

  Aakash Verma•6,710

  @skyv26Posted almost 3 years ago

  Very welldone, I saw your work @rizahoemae .

  You made it responsive too. But you did it uncomplete. I mean your hover states are absent.

  Now come to your question out of curiosity.

  When we come to take decision on choosing color then all have unique idea, choice. So there is no sure short answer for this but one thing keep in mind. If all thing is provided in the requirement then you have to do according to that requirement.

  Mobile response and style. Sure, You should always make your design persistant through out whole screen-width design. Even it color must be consistent too.

  Overall nice but never do uncomplete work (My advice)

  Good Luck For Future

  1

  ---

  

  Mawaddatur Riza•20

  @rizahoemaePosted almost 3 years ago

  @skyv26 hi thanks for visiting my work sir also thank you for reminding me of hover states. I forgot that, will fix it soon! :)

  1