NFT preview card component

Community feedback

• 

  Daniel 🛸•44,310

  @danielmrz-devPosted 11 months ago

  Hello @OsmanSamar!

  Your project looks great!

  I have a few suggestions that might interest you:

  • You can replace that h2 with h1 since it's the main title on the screen.

  The <h1> to <h6> tags are used to define HTML headings.

  <h1> defines the most important heading. <h6> defines the least important heading.

  Only use one <h1> per page - this should represent the main heading/subject for the whole page. Also, do not skip heading levels - start with <h1>, then use <h2>, and so on.

  • Also, you might wanna change the .card-description p color to a lighter one. Currently it's too dark, and since the bakcground is dark too, it's almost impossible to see it.

  I hope it helps!

  Other than those little details, you did a great job!

  Marked as helpful

  0