hitmorecode• 6,270
@hitmorecode
Posted
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