NFT Preview Card

Community feedback

• 

  Hania B.•1,320

  @techantherePosted almost 3 years ago

  Hey @cornthecob, a very nice effort for your first solution on Frontend Mentor, well done! I have a few things to mention here, which you can improve in future projects as well:

  1. Consider using a semantic tag like main and enclose all the content within it, while for attribution etc. you should add a footer tag.
  2. Try using at least one h1 level heading in the text. The text "Equilibrium" should be within the h1 tag.
  3. Always validate your code if it contains any missing opening or closing tag or other syntax-based mistakes in HTML, you can use w3c validator for this purpose.

  One thing more, I don't see the image hover effect in your solution, please try adding one, you will definitely learn a lot more along the process :) Good luck with your future solutions :)

  Marked as helpful

  1

  ---

  

  Cobb•20

  @cornthecobPosted almost 3 years ago

  Hey @techanthere, thank you very much for your feedback. They have undoubtedly helped me a lot. I've altered the solution to reflect all of your suggestions.

  Thanks once again! 🙏🏻

  1