Design comparison
SolutionDesign
Solution retrospective
Hi, My second project as a newbie. Any advice or suggestion Would be appreciated!
Community feedback
- @hardy333Posted about 3 years ago
- use more vertical padding on button.
- Try to fix accessibility issues:
- use
<main></main>
element for.container
not dev - use h1 tag instead of h3
- use
Marked as helpful4 - @FluffyKasPosted about 3 years ago
Hiya, your solution looks really good! I have a few suggestions though:
-
You could add a
background-size: contain
to your background image so it covers the whole screen. -
The payment button could use a bit more padding top and bottom.
-
You could swap the container
div
tomain
to make it more semantic.
Marked as helpful1 -
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