Intro-section-with-dropdown-navigation

Community feedback

• 

  Marija Havaic•150

  @marijahavaicPosted almost 2 years ago

  Looks great! I agree, closing the menu would be even better. You could maybe consider the hover effect on desktop dropdown menus, from a UX perspective I think they are a much better choice.

  I have noticed that the .nav-slide has a red border and .aside-wrapper slides in when you open the menu. Not sure if this was intentional or not (I used Google Chrome as a browser). :)

  Marked as helpful

  1

  ---

  

  yishak abrham•2,150

  @yishak621Posted almost 2 years ago

  @marijahavaic o ya u r right the red border i use when i design to see that element cover space and i forgot to remove it when i finish ...thank u i will remove that ...and also i will add the hover effect too ..but i don't get what u mean on the aside wrapper...would u Please elaborate that 🙏🙏

  0

  

  Marija Havaic•150

  @marijahavaicPosted almost 2 years ago

  @yishak621 Sorry, I didn't explain it well. When you click on the menu icon the .aside-wrapper slides in with the overlay (background-color: rgba(0, 0, 0, 0.35)). Usually, the overlay doesn't transition, only the .nav-slide. Hope I explained it better now :)

  Marked as helpful

  0

  

  yishak abrham•2,150

  @yishak621Posted almost 2 years ago

  @marijahavaic o sure thanks its clear now the overlay doesnt need the slide behavior it just be appeared as it is ...noted👍

  0
• 

  👾 Ekaterine Mitagvaria 👾•7,600

  @catherineisonlinePosted almost 2 years ago

  Look great. Yes, it's even better if the menu closes after clicking the link because then the user would have to manually keep closing it all the time. I would also make the load animation of the dropdown a little faster, it's way too slow. Also make sure to add alt to the images to remove the report issues

  1

  ---

  

  yishak abrham•2,150

  @yishak621Posted almost 2 years ago

  @catherineisonline thanks for the response ...i will make the animation timing faster ...and also since i load the icons as image it says add alt but icons dont need alt text for screen readers ...or maybe i will load directly icons from font awesome 👍

  0

  

  👾 Ekaterine Mitagvaria 👾•7,600

  @catherineisonlinePosted almost 2 years ago

  @yishak621 They don't need alt but as I mentioned it can be done to remove the report issues and then changed back again 😎

  1