NFT component

Community feedback

• 

  Daniel 🛸•44,310

  @danielmrz-devPosted 9 months ago

  Hello @believer-danish!

  Your solution looks great!

  I have a suggestion for improvement:

  • Use <main> to wrap the whole main content instead of just the text. This tag is meant for the main content, including images, videos or any other elements that are part of it.

  This tag change does not impact your project visually and makes your HTML code more semantic, improving SEO optimization as well as the accessibility of your project.

  I hope it helps!

  Other than that, great job!

  Marked as helpful

  0

  ---

  

  Muhammad Danish•200

  @believer-danishPosted 9 months ago

  @danielmrz-dev thanks for your suggestion. I will try to use semantic elements more efficiently.

  1
• 

  Olaniyi Ezekiel•7,580

  @Ezekiel225Posted 9 months ago

  Hello there 👋 @believer-danish.

  Good job on completing the challenge !

  Your project looks really good!

  I have a suggestion about your code that might interest you.

  There is an very useful browser extension called Perfect Pixel that allow you compare with the design image and thus see the exact dimensions. I recommend it to you.

  I hope this suggestion is useful for future projects.

  Keep up the excellent work and continue to challenge yourself with new projects. Your progress is impressive, and each project is a step forward in your front-end development journey! 🚀🌟.

  Other than that, great job!

  Happy coding.

  Marked as helpful

  0

  ---

  

  Muhammad Danish•200

  @believer-danishPosted 9 months ago

  @Ezekiel225 Thanks, I didn't knew about Perfect Pixel. Surely i will use it.

  0