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

Mobile Menu, all but first child #627

Closed
bradymwilliams opened this issue Dec 14, 2015 · 5 comments
Closed

Mobile Menu, all but first child #627

bradymwilliams opened this issue Dec 14, 2015 · 5 comments

Comments

@bradymwilliams
Copy link

Hi there! Just started from a new install and I have 7 Custom Links all on the same level, but in the mobile menu only the first one is on the correct level. The rest of the links are indented.
Here is what the output is vs the builder.

image

What am I doing wrong?

@beniven
Copy link

beniven commented Dec 14, 2015

Having the same problem

@beniven
Copy link

beniven commented Dec 14, 2015

Hey @BlackSheepWithMe, it looks like the ul of the mobile navigation on line 46 of /library/navigation.php wasn't closed.

Should look like this:
'items_wrap' => '<ul id="%1$s" class="%2$s show-for-small-only" data-accordion-menu>%3$s</ul>',

@bradymwilliams
Copy link
Author

@beniven Thanks!

I'll leave this open so we can actually get it resolved in the repo

@erwinpagulong
Copy link

@beniven it works! Thanks a lot.

@colin-marshall
Copy link
Collaborator

Good find @BlackSheepWithMe and thanks for the fix @beniven! PR submitted.

erwinpagulong added a commit to erwinpagulong/FoundationPress that referenced this issue Dec 15, 2015
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

4 participants