Design comparison
Solution retrospective
Hello, What's up?!✌️
This is a update, this time I decrease the amount of classes used and applied css grid concepts. More than that, a hover was added on the button.
Any Feedback Is Greatly Appreciated!
Community feedback
- @VCaramesPosted about 2 years ago
Hey there! 👋 Here are some suggestions to help improve your code:
-
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.
-
The “30-day, hassle-free money back guarantee” is not a heading. It should instead be wrapped in a Paragraph Element.
-
The “Why Us” list needs to be created using an Unordered List Element along with the List Items Element.
-
Remove all the Line Break Elements you added, they are unnecessary.
If you have any questions or need further clarification, let me know.
Happy Coding! 👻🎃
Marked as helpful0 -
Please log in to post a comment
Log in with GitHubJoin our Discord community
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord