@RikvanderSar
Posted
Hi Adebayo,
Look good to me! If I compare your version with the design it looks like the inactive and active state of the links are vice versa. And I wonder why you've used a psuedo element for the background image. Wouldn't a background image on the body work as well and be less code?
Marked as helpful
@Abduwaasi
Posted
Thanks @RikvanderSar I really appreciate the time you took to review my code.
This is my first time working on a design, I usually follow tutorial video, to be honest I didn't understand what the active state mean in the design till I see your review, I will implement the correction ASAP.
For the background image I actually thought about inserting it in my background section but I was confused on how I will make the image appear on top of the background color. That's why I used pseudo element and gave it z-index
