Profile card component

Community feedback

• 

  バレンタイン 😈•64,170

  @VCaramesPosted about 2 years ago

  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-image property.

  • 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

  1

  ---

  

  Dr-Vegapunk007•140

  @Dr-Vegapunk007Posted about 2 years ago

  @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👌👌

  0

  

  バレンタイン 😈•64,170

  @VCaramesPosted about 2 years ago

  @Dr-Vegapunk007

  It's the rule when using alt tag description.

  https://accessibility.huit.harvard.edu/describe-content-images

  https://medium.com/finn-no/alt-text-for-images-11812900902a

  https://moz.com/learn/seo/alt-text

  1