@VCarames
Posted
Hey there! 👋 Here are some suggestions to help improve your code:
-
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.
-
The background images should be applied using CSS not HTML. You can accomplish this by using the
background-imageproperty. -
For profile image’s Alt Tag, you never use the keywords “image”, “photo” or “picture”.
-
“Victor Crest” is a heading. So it needs to be wrapped in a Heading Element.
If you have any questions or need further clarification, let me know.
Happy Coding! 👻🎃
Marked as helpful
@Dr-Vegapunk007
Posted
@vcarames, Thank you for helping I'm so grateful
Can I ask you why I shouldn't use the keywords "image photo or picture" in alt attribute
Happy Coding bro👌👌
@VCarames
Posted
@Dr-Vegapunk007
It's the rule when using alt tag description.
https://accessibility.huit.harvard.edu/describe-content-images
