CSS FLEX

Community feedback

• 

  Daniel•170

  @Catalyst497

  Posted about 3 years ago

  Wow, I love this solution.

  I just think if you were using flexbox for your navbar. You could add an align-items: center to it to make the items vertically centered.

  You can find my solution here: https://www.frontendmentor.io/solutions/flexbox-plenty-of-grid-layout-_rQIc3AANQ

  10

  Marked as helpful
• 

  Tesla_Ambassador•3,070

  @tesla-ambassador

  Posted about 3 years ago

  Hey Shekinah007. Congrats on completing this wonderful solution. I really like how responsive it is and the exit animations of the desktop nav at a significantly smaller screen width! It's really great! A few things you might want to consider:

  • I'd suggest that you add some padding a top your nav bar because it's a little too close to the edge. You might also want to trigger the drop down menu with a click but in the case of having to trigger it with hover, you should keep the drop down menu such that it doesn't disappear when a user hovers away from the link that triggered it. It'd be great if it disappears when you click one of the sub-links. Because you are using REACT I suggest that you use the onMouseEnter event to conditionally render the drop down menu and then a click event on one of the liks to change the conditions to false and stop rendering the element.
  • Inorder to fix some of your accessibility issues, you might want to add landmarks (<main>, <footer> etc.) to your page so that the browser can easily navigate your site. If you want to know more on landmarks, follow this link.
  • You should consider using divs as parent elements of ul or another div as a parent element of another div because it's not valid HTML for span to be a parent to any of those. This will fix your HTML validation issues. Happy coding and keep up the good work, you're doing really great!

  11

  Marked as helpful