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

Fix bug where toggling the menu on mobile would jump to the top of the page #400

Merged
merged 1 commit into from
Jan 2, 2022

Conversation

robbieaverill
Copy link
Contributor

No description provided.

@shalvah shalvah merged commit 51196b0 into knuckleswtf:master Jan 2, 2022
@shalvah
Copy link
Contributor

shalvah commented Jan 2, 2022

Not sure what this does exactly, but hope it works🤣

@robbieaverill robbieaverill deleted the bugfix/mobile-nav-menu branch January 5, 2022 00:48
@robbieaverill
Copy link
Contributor Author

Hey @shalvah, what it's doing is preventing the default behaviour when you click on the "menu" button on mobile views. The button is an anchor, so the default behaviour is to follow the href attribute, which is #. What you saw before this patch is that clicking on the menu button on mobile would open the menu, and jump to the top of the page. It now only opens the menu.

@shalvah
Copy link
Contributor

shalvah commented Jan 5, 2022 via email

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.

2 participants