I noticed you didn't do a hover effect for the links.
I think the correct way would be to use :hover, and make the links/li buttons and just styling them a specific way and hover to change the text to black and background to green.
I also would have used ul/li and class names such as list-items on parent and list-item/link-item on the children as opposed to P.
Good first run through though.
@fortunethecoder
Posted
thanks
@JoeWebDevelopment