Merge pull request #7 from rachelbaker/master

Added conditional for appended CSS

jquery.horizontalNav.js

@@ -6,47 +6,51 @@
  * Author URL: http://sebnitu.com
  */
 (function($) {
-    
+
     $.fn.horizontalNav = function(options) {
-                
+
         // Extend our default options with those provided.
         var opts = $.extend({}, $.fn.horizontalNav.defaults, options);
-        
+
         return this.each(function () {
-        
+
             // Save our object
             var $this = $(this);
-            
+
             // Build element specific options
             // This lets me access options with this syntax: o.optionName
             var o = $.meta ? $.extend({}, opts, $this.data()) : opts;
-            
+
             // Save the wrapper. The wrapper is the element that
             // we figure out what the full width should be
             if ($this.is('ul')) {
                 var ul_wrap = $this.parent();
             } else {
                 var ul_wrap = $this;
             }
-
-            // let's append a clearfixing element to the ul wrapper
-            ul_wrap.css({ 'zoom' : '1' }).append('<div class="clearHorizontalNav">');
-            $('.clearHorizontalNav').css({
+
+            if ($('.clearHorizontalNav').length ) {
+                ul_wrap.css({ 'zoom' : '1' });
+            } else {
+                ul_wrap.css({ 'zoom' : '1' }).append('<div class="clearHorizontalNav">');
+                // let's append a clearfixing element to the ul wrapper
+                $('.clearHorizontalNav').css({
                 'display' : 'block',
                 'overflow' : 'hidden',
                 'visibility' : 'hidden',
                 'width' : 0,
                 'height' : 0,
                 'clear' : 'both'
-            });
-
+                });
+            }
+
             // Grab elements we'll need and add some default styles
             var ul = $this.is('ul') ? $this : ul_wrap.find('> ul'), // The unordered list element
                 li = ul.find('> li'), // All list items
                 li_last = li.last(), // Last list item
                 li_count = li.size(), // The number of navigation elements
                 li_a = li.find('> a'); // Remove padding from the links
-            
+
             // If set to responsive, re-construct after every browser resize
             if ( o.responsive === true ) {
                 // Only need to do this for IE7 and below
@@ -55,22 +59,22 @@
                     resizeTrigger( _construct, o.responsiveDelay );
                 }
             }
-            
+
             // Initiate the plugin
             _construct();
-            
+
             // Returns the true inner width of an element
             // Essentially it's the inner width without padding.
             function trueInnerWidth(element) {
-                return element.innerWidth() - ( 
-                    parseInt(element.css('padding-left')) + parseInt(element.css('padding-right')) 
+                return element.innerWidth() - (
+                    parseInt(element.css('padding-left')) + parseInt(element.css('padding-right'))
                 );
             }
-            
+
             // Call funcion on browser resize
             function resizeTrigger(callback, delay) {
                 // Delay before function is called
-                delay = delay || 100;   
+                delay = delay || 100;
                 // Call function on resize
                 var resizeTimer;
                 $(window).resize(function() {
@@ -80,36 +84,36 @@
                     }, delay);
                 });
             }
-            
+
             // The heavy lifting of this plugin. This is where we
             // find and set the appropriate widths for list items
             function _construct() {
-            
+
                 if ( (o.tableDisplay != true) || ($.browser.msie && parseInt($.browser.version, 10) <= 7) ) {
-                    
+
                     // IE7 doesn't support the "display: table" method
                     // so we need to do it the hard way.
-                    
+
                     // Add some styles
                     ul.css({ 'float' : 'left' });
                     li.css({ 'float' : 'left', 'width' : 'auto' });
                     li_a.css({ 'padding-left' : 0, 'padding-right' : 0 });
-                    
+
                     // Grabbing widths and doing some math
                     var ul_width = trueInnerWidth(ul),
                         ul_width_outer = ul.outerWidth(true),
                         ul_width_extra = ul_width_outer - ul_width,
-                        
+
                         full_width = trueInnerWidth(ul_wrap),
                         extra_width = (full_width - ul_width_extra) - ul_width,
                         li_padding = Math.floor( extra_width / li_count );
-                    
+
                     // Cycle through the list items and give them widths
                     li.each(function(index) {
                         var li_width = trueInnerWidth( $(this) );
                         $(this).css({ 'width' : (li_width + li_padding) + 'px' });
                     });
-                    
+
                     // Get the leftover pixels after we set every itms width
                     var li_last_width = trueInnerWidth(li_last) + ( (full_width - ul_width_extra) - trueInnerWidth(ul) );
                     // I hate to do this but for some reason Firefox (v13.0) and IE are always
@@ -119,23 +123,23 @@
                     }
                     // Add the leftovers to the last navigation item
                     li_last.css({ 'width' : li_last_width + 'px' });
-                
+
                 } else {
                     // Every modern browser supports the "display: table" method
                     // so this is the best way to do it for them.
                     ul.css({ 'display' : 'table', 'float' : 'none', 'width' : '100%' });
                     li.css({ 'display' : 'table-cell', 'float' : 'none' });
                 }
             }
-        
+
         }); // @end of return this.each()
 
     };
-    
+
     $.fn.horizontalNav.defaults = {
         responsive : true,
         responsiveDelay : 100,
         tableDisplay : true
     };
-    
+
 })(jQuery);