Design comparison
SolutionDesign
Community feedback
- @MelvinAguilarPosted almost 2 years ago
Hello there ๐. Good job on completing the challenge !
I have some suggestions about your code that might interest you.
HTML ๐:
- Use the
<main>
tag to wrap all the main content of the page instead of the<section>
tag. With this semantic element you can improve the accessibility of your page.
- It is wrong to create a div for the "Proceed to Payment" button, if I as a user click anywhere on the blue "button", nothing will happen because the button is the text, the rest is a div. You should style the button like the div and not have a button inside a div.
I hope you find it useful! ๐
Happy coding!
Marked as helpful1@brunafieldsPosted almost 2 years ago@MelvinAguilar Thank you for the feedback!
I'm just starting to do a few projects on my own. I guess the real learning process is happening now.
Thank you. ๐
1 - Use the
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