search

sebnitu / HorizontalNav

A jQuery plugin that spans a horizontal navigation to fit the full width of it's container.
MIT License
277 stars 43 forks source link

.clearHorizontalNav div gets duplicated easily #6

Closed rachelbaker closed 11 years ago

rachelbaker commented 11 years ago

Thank you for all of your work on HorizontalNav, it is a great menu script. I often use HorizontalNav in conjunction with another js script like TinyNav and I found that when the window is resized the div .clearHorizontalNav gets created with each resize.
See screenshot here: http://cl.ly/image/363Y2L15411i

This looks to be a result of the append('<div class="clearHorizontalNav">'); not running only if it doesn't already exist.

sebnitu commented 11 years ago

Thank you for finding this! Should be an easy fix, will definitely patch it on the next update.

rachelbaker commented 11 years ago

I submitted my fix in a pull request here: https://github.com/sebnitu/HorizontalNav/pull/7

Again, thanks for your work on this!

sebnitu commented 11 years ago

Awesome, thank you for the fix! Much appreciated :D