@VCarames
Posted
Hey there! 👋 Here are some suggestions to help improve your code:
Regarding your question about the transition,
You can always add a third layout. That’s what I did with my project;
https://www.frontendmentor.io/solutions/social-proof-section-css-flex-and-animation-YPeDVv5Qs6
- The `headers is not needed for this challenge, since it is a section of a larger site (as the title of the challenge states).
- Replace the
figure
with adiv
in star sections as they do not have afigcaption
.
- 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). These testimonials cannot do neither.
More Info:📚
- To improve the semantics of your component, you will want to wrap each individual testimonial component in a
figure
element, the individuals information should be wrapped in afigcaption
element and lastly, the testimonial itself should be wrapped in ablockquote
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