hardy• 3,660
@hardy333
Posted
- use more vertical padding on button.
- Try to fix accessibility issues:
- use
<main></main>element for.containernot dev - use h1 tag instead of h3
- use
Marked as helpful
4
Design comparison
Solution retrospective
Hi, My second project as a newbie. Any advice or suggestion Would be appreciated!
Community feedback
P
Fluffy Kas• 7,735
@FluffyKas
Posted
Hiya, your solution looks really good! I have a few suggestions though:
-
You could add a
background-size: containto 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
divtomainto make it more semantic.
Marked as helpful
1
Please log in to post a comment
Log in with GitHub
Join 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