Added support for not processing a menu #8

Merged
merged 2 commits into from Sep 4, 2012

Conversation

Projects
None yet
2 participants
@Hambrook
Contributor

Hambrook commented Sep 4, 2012

Either because it has too few items or because JS isn't working.

Hambrook added some commits Sep 4, 2012

Update jquery.horizontalNav.js
Added: Option ```minimumItems``` to specify the minimum number of menu items required before setting the width. This should avoid having one or two items go super wide.
Also the class ```.horizontalNav-processed``` will be added to the supplied element if horizontalNav does is used, and ```.horizontalNav-notprocessed``` will be added if it's not. Useful for manually adding padding for when ```minimumItems``` is used. Also consider adding ```.horizontalNav-notprocessed``` to your raw html code to trigger padding in case of JS not loading.
Update jquery.horizontalNav.js
Fixed: Was adding processed/notprocessed class to parent element, not supplied element. Also fixed ```.horizontalNav-notprocessed``` not being removed if hardcoded via html for non JS fallback..

sebnitu added a commit that referenced this pull request Sep 4, 2012

Merge pull request #8 from Hambrook/master
Added support for not processing a menu

@sebnitu sebnitu merged commit c4ee7a1 into sebnitu:master Sep 4, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment