NFT-Card

Community feedback

• 

  Adriano•38,080

  @AdrianoEscarabotePosted about 2 years ago

  Hi Huey.io, how are you?

  You've been doing a great job here at fem, keep coding, I guarantee that a month from now you'll be glad you didn't stop!

  I really liked the result of your project, but I have some tips that I think you will enjoy:

  • to improve the design of the page, we can do this:

  body {
      margin: 0 auto;
  }
  

  The rest is great!

  I hope it helps... 👍

  Marked as helpful

  1
• 

  バレンタイン 😈•64,190

  @VCaramesPosted about 2 years ago

  Hey there! 👋 Here are some suggestions to help improve your code:

  Here is a link regarding the image overlay:

  https://www.w3schools.com/howto/howto_css_image_overlay.asp

  • The purpose of the Main Element is to identify the main content of your page. It is not the container of you component. After the main element, you want add a container to wrap you separate components in.

  • Add a border-radius to the component's container and NFT image to better match the FEM design.

  • The NFT Alt Tag description needs to be improved upon. You want to describe what the image is; they need to be readable. Assume you’re describing the image/icon to someone.

  • Wrap the "NFT image" and "Equilibrium #3429" in an Anchor Tags <a>. The anchor tag will allow users to click on content and have them directed to another part of your site.

  If you have any questions or need further clarification, let me know.

  Happy Coding! 👻🎃

  Marked as helpful

  1