@VCarames
Posted
Hey there! 👋 Here are some suggestions to help improve your code:
- 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.
-
There are three headings in this component; “Join our community”, “Monthly Subscription”, and “Why us”. They need be wrapped in a header element.
-
The button was created with the incorrect element. When users click on the button they should directed to a different part of your site; that Anchor Element will allow this to happen.
-
The “Why Us” list needs to be created using an Unordered List Element along with the List Items Element.
If you have any questions or need further clarification, let me know.
Happy Coding! 👻🎃
Marked as helpful
@LeonardoSchuquel
Posted
Hi @vcarames I confess that I didn't quite understand the first tip about the main and container but I tried to make some changes to them. If it's still not good weight better explanation
@VCarames
Posted
@LeonardoSchuquel
So essential your HTML code should look like this:
<body>
<main>
<div class="components-container">
<div class="white-section"></div>
<div class="cyan-secton"></div>
<div class="light-cyan-section"></div>
</div>
</main>
</body>
You'll notice that after the main element theres a container class and inside that are all the individual components.
Marked as helpful
