News homepage with responsive navbar

Community feedback

• 

  pankaj ram pawar•380

  @pankajrampawarPosted 12 months ago

  hey Aayush, for the backdrop use a position-fixed div with screen height and width, reduce its opacity and give it a light grey or slate colour, make sure it has a Index higher than page but lower than sidebar. toggle the visibility of the backdrop based on visibility of the sidebar. and there you go.. I hope you find this helpful.

  Marked as helpful

  0

  ---

  

  Aayush Jha•410

  @Aayush895Posted 11 months ago

  @pankajrampawar Hey, thanks for the suggestion. I understand what to do now. So thanks for taking the time out and dropping the suggestion. I appreciate it.

  0
• 

  Andrés Gutiérrez Ramírez•2,470

  @AGutierrezRPosted 12 months ago

  Hello there! 👋 Great job on completing the challenge!

  I have some suggestions regarding your code that might be of interest to you.

  When working with animations to show and hide elements, the display rule primarily aids in displaying elements. If an element's display is not set to none, the animation will activate, displaying the element. However, setting display: none hides the element before the exit animation can take effect.

  To address this issue, consider adding a new class, something--closing, specifically designed to animate the element's exit. You can then utilize the animationend event to wait for this animation to conclude before applying the display: none rule.

  I hope you find this suggestion helpful! 😁 Your submitted solution is fantastic!

  Happy coding!

  Marked as helpful

  0

  ---

  

  Aayush Jha•410

  @Aayush895Posted 11 months ago

  @AGutierrezR Hey, thanks for taking the time out and dropping your suggestion. I appreciate it, and your suggestion gave me some understanding of what I was doing wrong. So thank you once again. Have a great day.

  0