NFT Card Component Project

Community feedback

• 

  BasemAmr•120

  @BasemAmrPosted almost 3 years ago

  Greetings Aidan, congratulations on finishing the first challenge 👏

  I have some small notes that may be useful,

  • you can replace "article" elements with divs, as they aren't articles, they are, like, small containers to layout the card.

  • the container "div" can be a "main" element to solve the accessibility issue

  • the equilibrium big image 'alt' text should be assigned to sometext describes it

  • finally, set a width for the card and center it, at the end, it's a card, a part from a landing page, for example, that takes some width of the page, but not all width.

  and that's it, I hope that these notes can be helpful

  Best of luck, happy coding 😀

  Marked as helpful

  0