
Diplay Flex / Display Grid / Media Queries
Design comparison
Community feedback
- @VCaramesPosted over 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 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 helpful0@HaciellpPosted over 2 years ago@vcarames Thank you very much brother, thanks for the suggestion, a question, then the button should be created with a button type input?
0@VCaramesPosted over 2 years ago@Haciellp
It depends on what action you want your "button" to do.
If you want to add something to a cart or submit a form, then a Button Element would be the best choice.
But if you want to direct your users to another part of your site or to another website, etc... then the Anchor Tag would be the better option.
Marked as helpful0@HaciellpPosted over 2 years ago@vcarames oooh I understand, absolutely right, so since this case is a sign up button, yes it should go with an anchor tag. Thank you!!!!
0 -
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