Skip to content

Fixes clicking on menu bar sub-menu entries#337

Merged
Keavon merged 3 commits into
masterfrom
menufix
Aug 10, 2021
Merged

Fixes clicking on menu bar sub-menu entries#337
Keavon merged 3 commits into
masterfrom
menufix

Conversation

@KeatonTech

@KeatonTech KeatonTech commented Aug 10, 2021

Copy link
Copy Markdown
Contributor

Previously the parent menu would close on MouseDown, before the Click event could trigger on submenu items. This fix involves ensuring that the parent menu stays open as long as the cursor remains on any of its sub-menus.

Fixes #280


This change is Reviewable

…e closed before the submenu click could be registered. Also ignores package-lock.json.
@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Aug 10, 2021

Copy link
Copy Markdown

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: a264d14
Status: ✅  Deploy successful!
Preview URL: https://eb9f4ac7.graphite-master.pages.dev

View logs

@Keavon Keavon merged commit 30a4159 into master Aug 10, 2021
@Keavon Keavon deleted the menufix branch August 10, 2021 21:54
Keavon pushed a commit that referenced this pull request Jun 16, 2022
* Fix a bug where clicking a submenu would cause the parent menu to be closed before the submenu click could be registered. Also ignores package-lock.json.

* Remove package-lock gitignore

* Add back empty line
Keavon pushed a commit that referenced this pull request Jul 30, 2023
* Fix a bug where clicking a submenu would cause the parent menu to be closed before the submenu click could be registered. Also ignores package-lock.json.

* Remove package-lock gitignore

* Add back empty line
@Keavon

Keavon commented Jun 13, 2026

Copy link
Copy Markdown
Member

Hi @KeatonTech, thanks again for this code contribution to the project! We're still hoping you will respond to the request to relicense this code. Please see #4208 ASAP, thank you!

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Clicking on menu bar sub-menu entries doesn't work

2 participants