P
Franklin Gomez• 160
@frankgomezdev
Posted
My advice and feedback would be:
- Be a little more descriptive with your class names.
- Try using an <a> tag for the links seeing as they can be clicked and would direct you to a different page.
- The color change to yellow should be on the ":hover" state of the links instead of active. (I might be wrong about this, gotta go double check the design).
- Take a look into using Flexbox or CSS Grid. Once you understand it, it's way easier to create this layout without having to use fixed heights and widths and pay attention to the padding and spacing between elements.
Great job and keep up the good work!
Marked as helpful
1
