Skip to content
This repository has been archived by the owner on Sep 16, 2019. It is now read-only.

Off canvas mobile menu issue #709

Closed
cotswold opened this issue Feb 5, 2016 · 5 comments
Closed

Off canvas mobile menu issue #709

cotswold opened this issue Feb 5, 2016 · 5 comments

Comments

@cotswold
Copy link

cotswold commented Feb 5, 2016

I've created a main menu and see there are 2 positions available, top right and mobile so I've assigned it to top right. Now when I make the browser window small the hamburger icon appears and the menu disappears from top-right as you'd expect however if you click the hamburger icon nothing happens.

I thought it might just be that I need to create another menu and assign to mobile position so did that but now the second menu displays under the top right menu in a list. If I make the browser smaller both menus disappear but the hamburger still doesn't work. I've checked the console and am not getting any error notices.

screen shot 2016-02-05 at 14 27 56

I'd be very grateful if anybody knows why it isn't working? I have the latest files installed.

Could it be related to the file path problems? In order to get the npm gulp install/build to work without errors I had to change the gulpfile.js paths from assets/components to bower_components

@mlangone
Copy link

mlangone commented Feb 5, 2016

Have you tried this?
#639 (comment)

@cotswold
Copy link
Author

cotswold commented Feb 5, 2016

That worked thanks! I had been trying to set it up in the admin area without realising the mobile menu setting is in customizer

@colin-marshall
Copy link
Collaborator

@cotswold most likely those are file permission problems you were having, not file path problems. assets/components is the path for bower which is set in .bowerrc. You should sort out the errors instead of changing all the paths to bower_components. What errors were you experiencing during npm install and npm run build?

If it's a permissions issue, see:
https://docs.npmjs.com/getting-started/fixing-npm-permissions

@olefredrik
Copy link
Owner

Just discovered that a few errors with the topbar was introduced in my latest navigation changes. Should be fixed now #714 .

@FlatspinZA
Copy link

I can confirm that the issue with the menu title is now fixed.

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

No branches or pull requests

5 participants