@VCarames
Posted
Hey there! 👋 Here are some suggestions to help improve your code:
-
The background image are not showing.
-
To center you content to your page, add the following to your Body Element:
body {
min-height: 100vh;
display: grid;
place-content: center;
}
-
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.
-
Remove al the
<br>
elements you added. They are unnecessary. -
The the figure element is being used incorrectly, instead wrap the "stars" and "reviews" in a Div Element.
-
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.
-
The profile images Alt Tags need to be improved. It should state the following; “Headshot of -person’s full name-“
-
To enhance the semantics of your component, you 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:
- Implement a Mobile First approach 📱 > 🖥
With mobile devices being the predominant way that people view websites/content. It is more crucial than ever to ensure that your website/content looks presentable on all mobile devices. To achieve this, you start building your website/content for smaller screen first and then adjust your content for larger screens.
If you have any questions or need further clarification, let me know.
Happy Coding! 👻🎃