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

Allow user to change breakpoint for mobile menu #45040

Open
Nyiriland opened this issue Oct 17, 2022 · 3 comments · May be fixed by #45274
Open

Allow user to change breakpoint for mobile menu #45040

Nyiriland opened this issue Oct 17, 2022 · 3 comments · May be fixed by #45274
Labels
[Block] Navigation Affects the Navigation Block [Type] Enhancement A suggestion for improvement.

Comments

@Nyiriland
Copy link

Nyiriland commented Oct 17, 2022

Issue

Currently, the default breakpoint for the mobile menu is under 600px. This means that desktop header navs with long menus generally wrap in a very unattractive way before the breakpoint kicks with a hamburger button. See exaggerated example:

image

This is one thing I almost always end up overriding with the CSS customizer.

Suggestion

It would be great to allow the user to add their own breakpoint for the mobile menu. I think it would make sense to add this to the Navigation block, under the Display section, if the "Mobile" option is chosen. Quick mockup below.

image

@kathrynwp kathrynwp added [Type] Enhancement A suggestion for improvement. [Block] Navigation Affects the Navigation Block labels Oct 17, 2022
@kathrynwp
Copy link

related: #42305

@getdave getdave linked a pull request Oct 25, 2022 that will close this issue
6 tasks
@Azragh
Copy link

Azragh commented Nov 7, 2022

#13363
#20244
#35808
#35848
#40870

@helixgdn
Copy link

helixgdn commented Sep 13, 2023

I posted a solution here:

#54388 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Block] Navigation Affects the Navigation Block [Type] Enhancement A suggestion for improvement.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants