NFT preview card component

Community feedback

• 

  bikeinman•1,080

  @BikeInManPosted almost 3 years ago

  I think you did a good job.

  Apart from the validator errors, I would like to recommend the following:

  1. Using header tags(h4 etc) for sizing is not recommended. Try to use p and adjust size in css for them.
  2. If you were to use flex on your .info , just like you did on .container you would have better alignment in an easier way.
  3. May be you can simplifying a bit more, but not adding classes at all to h3 etc., because they are easily accessible in css via selectors.
  4. Use divs only to group elements and to style as a group.

  Good Luck.

  Marked as helpful

  0

  ---

  

  Taha Khairy•60

  @tahakhairyPosted almost 3 years ago

  @BikeInMan Thank you very much for the feedback.

  0