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

Small issue with offcanvas menu #646

Closed
wishe opened this issue Dec 21, 2015 · 2 comments
Closed

Small issue with offcanvas menu #646

wishe opened this issue Dec 21, 2015 · 2 comments

Comments

@wishe
Copy link

wishe commented Dec 21, 2015

In "navigation.php", there is a small error in the build up of the menu. More specifically in the foundationpress_mobile_nav().

'items_wrap'     => '<ul id="%1$s" class="%2$s show-for-small-only" data-accordion-menu%3$s</ul>'
should be changed to
'items_wrap'     => '<ul id="%1$s" class="%2$s show-for-small-only" data-accordion-menu>%3$s</ul>'

This should fix the menu so that the first item also gets put in its own li and the ul gets closed correctly!

@olefredrik
Copy link
Owner

Hi,

This was fixed 6 days ago :)
f25bf3a

@wishe
Copy link
Author

wishe commented Dec 21, 2015

Oops my bad!

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

2 participants