@darryncodes
Posted
Hi Javier,
Glad to hear the designs are getting easier - keep going!
A few bits of feedback to help match the design a little more:
- add
min-height: 100vh;
to your<body>
and remove themargin top/bottom
on the.portafolio
class to centre the design nicely in the veiwport - add
background-color: hsl(0, 0%, 12%);
to.portafolio
, you'll need somepadding
andborder-radius
too - add
background-color: hsl(0, 0%, 20%);
to your buttons - use
cursor: pointer;
on the buttons to help indicate visually you can interact with them
A couple of pointers regarding semantic html:
- consider using a
<main>
for this element.portafolio
- try nesting your
a
's in aul
list - it's not valid HTML to wrap
button
's witha
tags. You probably just need the links and not the button on this design
I hope this helps. Happy coding 🤙