@VCarames
Posted
Hey there! 👋 Here are some suggestions to help improve your code:
- To not only improve your HTML code but to also identify the main content of you page, you will want to wrap your entire component inside the
main
element.
More Info:📚
- Remove* the
max-height: 700px;
from your component's container; it is not needed.
- 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 to someone.
- The “Icons” serve no other purpose than to be decorative; They add no value. Their
alt tag
should left blank and have anaria-hidden=“true”
to hides it from assistive technology.
- The profile image needs an Alt Tag. It should state the following; “Headshot of -person’s full name-“
- Wrap the "NFT image", "Equilibrium #3429" and "Jules Wyvern" in an
anchor tags
. 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, feel free to reach out to me.
Happy Coding! 🍂🦃
Marked as helpful
@ladaspcsn
Posted
@vcarames Thank you so much for the detailed feedback! I really appreciate it 😊