Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Smoother navigation bar UX to reduce flickering and missed clicks #316

Open
SableRaf opened this issue Aug 31, 2022 · 12 comments
Open

Smoother navigation bar UX to reduce flickering and missed clicks #316

SableRaf opened this issue Aug 31, 2022 · 12 comments

Comments

@SableRaf
Copy link
Collaborator

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/

@Dev-Code24
Copy link
Contributor

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

@SableRaf
Copy link
Collaborator Author

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

@Dev-Code24
Copy link
Contributor

Dev-Code24 commented Feb 14, 2023

@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

@SableRaf SableRaf assigned Dev-Code24 and unassigned runemadsen Feb 16, 2023
@SableRaf
Copy link
Collaborator Author

SableRaf commented Feb 16, 2023

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!

@Dev-Code24
Copy link
Contributor

Dev-Code24 commented Feb 24, 2023

@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 Dev-Code24 removed their assignment Feb 24, 2023
@SableRaf
Copy link
Collaborator Author

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

@Rahulhanje
Copy link
Contributor

@SableRaf is this issue fixed ?

@SableRaf
Copy link
Collaborator Author

SableRaf commented Aug 8, 2024

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

@Rahulhanje
Copy link
Contributor

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

@SableRaf
Copy link
Collaborator Author

SableRaf commented Aug 8, 2024

@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?

@Rahulhanje
Copy link
Contributor

Rahulhanje commented Aug 8, 2024

@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
Copy link
Contributor

@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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants