Design comparison
SolutionDesign
Community feedback
- @hitmorecodePosted 9 months ago
Nice well done. A few comments
- You only have a mobile version, the desktop version is missing
- You forgot to add the hover effect on the buttons
- You used <p> tag for the buttons link. Change it to <a> otherwise users will not be able to click on the links.
- You don't have a h1 on your page
- You can remove
<div class="background">
inside the CSS change.background
to body - You usde
position: relative;
to position the buttons, image and text. You can easily do this with flexbox or css grid. Only use position when necessary, otherwise just go with flexbox or css grid.
It's a good start Keep it up
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