@VCarames
Posted
Hey there! 👋 Here are some suggestions to help improve your code:
-
The Main Element should only wrap the card components. Since that is your main content.
-
The introductory text should be wrapped in a Header Element.
-
The “Reliable, efficient delivery Powered by Technology” is one single heading so the entire thing should be wrapped in a single <h1> Heading along with a Span Element.
-
The Article Element is not the best choice for wrapping these card components. In order to use the Article Element the components need to be able to make sense on its own and be independently distributable (can be used in on another site). These card components cannot do neither. Instead, you want to wrap each individual testimonial component in a Div and give them a class.
-
Add a third layout to make the transition from mobile 📱 -> desktop 🖥 views smoother.
If you have any questions or need further clarification, let me know.
Happy Coding! 👻🎃
Marked as helpful
@Hatchino
Posted
Hi @vcarames
- I add the header and the card components together with the main element because I wanted everything centered on the page
- I see what you mean to make a smoother transition but I don't know how can I do this Thank you for your help !
@VCarames
Posted
@Hatchino
The only purpose of the main is identify the main content of your site. You are using it as a container. Which is not the correct way of doing things.
If you goal is to center you content, then move all these properties to the Body Element:
main {
width: 100%; (delete)
height: 100vh; (change to `min-height`)
display: flex;
flex-direction: column;
align-items: center;
justify-content: center;
}