processing-website icon indicating copy to clipboard operation
processing-website copied to clipboard

Smoother navigation bar UX to reduce flickering and missed clicks

Open SableRaf opened this issue 2 years ago • 12 comments

In its current state, the dropdown menu on the navigation bar is prone to flickering, and it is sometimes hard to click the button you want to hit. If your cursor hovers just one pixel outside of the hitbox the menu disappears.

There are a few ways to solve this, from introducing a delay (with the issue that it creates lag) to smarter approaches that analyze mouse movement. Some more details and code examples can be found in this article: https://www.smashingmagazine.com/2021/05/frustrating-design-patterns-mega-dropdown-hover-menus/

SableRaf avatar Aug 31 '22 10:08 SableRaf

is this issue fixed? I cannot see any PR is made referring to this issue.

Dev-Code24 avatar Feb 14 '23 09:02 Dev-Code24

Hi @Dev-Code24, this hasn't been fixed yet. Note: this should probably be addressed together with issue https://github.com/processing/processing-website/issues/430 "Improve top bar menu navigation for keyboard and screen reader users"

SableRaf avatar Feb 14 '23 11:02 SableRaf

@SableRaf ok, I will fix both issues in a single PR. is it required to be assigned to issue or not? because the issue that I fixed earlier was also assigned to someone else 2 weeks back but he didn't do it

Dev-Code24 avatar Feb 14 '23 12:02 Dev-Code24

Thanks @Dev-Code24. I assigned this to you. It is not strictly required at the moment, but I think it helps keeping things tidy so I'm trying to get into the habit of assigning issues. Thanks for your contribution!

SableRaf avatar Feb 16 '23 14:02 SableRaf

@SableRaf Sorry I am unable to fix the issue at the moment so because of that I am unassigning myself so that others can work on the issue without facing any problem.

Dev-Code24 avatar Feb 24 '23 08:02 Dev-Code24

@Dev-Code24 no problem thanks for looking into it :) If you gathered any insights on the issue feel free to share them.

SableRaf avatar Feb 24 '23 11:02 SableRaf

@SableRaf is this issue fixed ?

Rahulhanje avatar Aug 08 '24 07:08 Rahulhanje

@Rahulhanje Not yet! Let me know if you'd like to take this on.

SableRaf avatar Aug 08 '24 11:08 SableRaf

Yes, sure Since I am a beginner I would be happy if you could guide me on this issue need some time

Rahulhanje avatar Aug 08 '24 12:08 Rahulhanje

@Rahulhanje This issue is a bit more advanced and I'm not sure we have someone available to mentor you through it. I would suggest you start with the issue we already assigned to you and come back to this after the other is resolved or once you have gained a bit more confidence with the codebase. How does that sound?

SableRaf avatar Aug 08 '24 12:08 SableRaf

@SableRaf you guessed it correctly i don't have anyone one to mentor me so first i will try to fix the first issue you are assigned to me then i will come to this issue I am ok with you

Rahulhanje avatar Aug 08 '24 12:08 Rahulhanje

@SableRaf "I've completed my first issue, and the PR has been merged. I'd like to give this new issue a try. Could you please assign it to me and provide some ideas on how to solve the error

Rahulhanje avatar Aug 12 '24 11:08 Rahulhanje