@VCarames
Posted
Hey there! 👋 Here are some suggestions to help improve your code:
- Along with the blank alt tag, you also want to include the aria-hidden=“true” to your stars to fully remove it from assistive technology.
More Info:📚
https://www.w3schools.com/html/html_images_picture.asp
- The profile images are not decorative. They need to haveAlt Tags; It should state the following; “Headshot of -person’s full name-“
- The
article
element is not the best choice for wrapping these testimonials. In order to use thearticle
element the component needs to be able to make sense on its own and be independently distributable (can be used in on another site).
You instead, want to wrap each individual testimonial component in a Figure
element, the individuals information should be wrapped in a Figcaption
element and lastly, the testimonial itself should be wrapped in a Blockquote
element.
Code:
<figure>
<figcaption></figcaption>
<blockquote></blockquote>
</figure>
More Info:
If you have any questions or need further clarification, feel free to reach out to me.
Happy Coding!🎄🎁
Marked as helpful
@Oluwatobi9034
Posted
@vcarames Thank you very much for your feedback. I really do appreciate.
@VCarames
Posted
@Oluwatobi9034
Glad I could help!
Keep it up!
@Oluwatobi9034
Posted
Hello @vcarames, can I add both the property aria-hidden and alt attribute ?
@VCarames
Posted
@Oluwatobi9034
For images that are solely decorative, yes.
The alt
would be left blank (alt=""
)
@Oluwatobi9034
Posted
@vcarames okay, thanks.