Responsive dropdown navigation homepage using Sass, html & javaScript

Community feedback

• 

  Julius Blezinger•180

  @JBleziPosted about 1 year ago

  Hi Gift! It looks great overall, really good work :)

  Apart from the advice already given by Matias, the only thing I would change is that you can make it so the menus close when you open another menu, so when the 'Features' menu is open and you click on the 'Company' menu, then the 'Features' menu closes. This helps the user focus better on the menu they just clicked on.

  By doing that it is also easier to align the dropdown with the menu item, because then the other dropdown won't be in the way ;)

  Hope this helps, keep up the good work!

  Marked as helpful

  1

  ---

  

  Gift Amachree•870

  @jen67Posted about 1 year ago

  @JBlezi Thank you so much

  1
• 

  Matias Ludueña•1,990

  @matiasluduena23Posted about 1 year ago

  Hi Gift Amachree’s! Nice work!!! Just one advice that might help you.

  • In desktop design the both side of the page looks like two boxes with the same width. One has the image, the other one the text. To approach this you can use grid or flex. Since you used flex let look at it with flex

  you have
  
  @media (min-width: 768px) {
    main {
      display: flex;
      gap: 2rem;
      flex-direction: row-reverse;
      padding-left: 3rem;
      padding-right: 3rem;
    }
  
  (and add)
  .main-section-texts, 
  .wrapper-figure {
  width: 50%;
  }
  (I thing is better to use a wrapper for you picture tag.) 
  
  

  We set the width of the children elements to 50%.

  Hope this help you!

  Good code!

  Matias

  Marked as helpful

  1