@dusan-b
Posted
Hi,
you did a good job building semantic and accessible HTML.
Regarding the rating stars, I think the only way to avoid repetition is to merge the SVG images into one image. However, you could also do something like the following:
// HTML
<div class="rated review">
<span class="star-icon"></span>
<span class="star-icon"></span>
<span class="star-icon"></span>
<span class="star-icon"></span>
<span class="star-icon"></span>
</div>
// CSS
.star-icon {
width: 17px;
height: 16px;
background: transparent url("../images/icon-star.svg") center no-repeat;
}
According to MDN, the <section>
element should usually contain a heading to indicate the content's purpose for SEO and assistive technologies. For the testamonials section I would consider replacing <section>
with <div>
, or alternatively adding a visually hidden <h2>
element as a direct child of <section>
.
Besides that, your code is well structured. Keep it up!
Marked as helpful