Improve naming and styling.

Author: LOBsTerr

superfish.js

@@ -14,7 +14,7 @@
  */
 
 (function($){
-  $.fn.superfish = function(op){
+  $.fn.superfish = function(options){
     var sf = $.fn.superfish,
       c = sf.c,
       $arrow = $(['<span class="',c.arrowClass,'"> &#187;</span>'].join('')),
@@ -26,11 +26,13 @@
       out = function(){
         var $$ = $(this), menu = getMenu($$), o = sf.op;
         clearTimeout(menu.sfTimer);
-        menu.sfTimer=setTimeout(function(){
+        menu.sfTimer = setTimeout(function(){
           if ($$.children('.sf-clicked').length == 0){
-            o.retainPath=($.inArray($$[0],o.$path)>-1);
+            o.retainPath = ($.inArray($$[0],o.$path)>-1);
             $$.hideSuperfishUl();
-            if (o.$path.length && $$.parents(['li.',o.hoverClass].join('')).length<1){over.call(o.$path);}
+            if (o.$path.length && $$.parents(['li.',o.hoverClass].join('')).length < 1){
+              over.call(o.$path);
+            }
           }
         },o.delay);
       },
@@ -43,8 +45,8 @@
 
     return this.each(function() {
       var s = this.serial = sf.o.length;
-      var o = $.extend({},sf.defaults,op);
-      o.$path = $('li.'+o.pathClass,this).slice(0,o.pathLevels);
+      var o= $.extend({},sf.defaults,options);
+      o.$path = $('li.' + o.pathClass,this).slice(0,o.pathLevels);
       var p = o.$path;
       for (var l = 0; l < p.length; l++){
         p.eq(l).addClass([o.hoverClass,c.bcClass].join(' ')).filter('li:has(ul)').removeClass(o.pathClass);
@@ -54,8 +56,7 @@
       $('li:has(ul)',this)[($.fn.hoverIntent && !o.disableHI) ? 'hoverIntent' : 'hover'](over,out).each(function() {
         if (o.autoArrows) addArrow( $(this).children('a:first-child, span.nolink:first-child') );
       })
-      .not('.'+c.bcClass)
-        .hideSuperfishUl();
+      .not('.' + c.bcClass).hideSuperfishUl();
 
       var $a = $('a, span.nolink',this);
       $a.each(function(i){
@@ -89,7 +90,7 @@
     pathClass: 'overideThisToUse',
     pathLevels: 1,
     delay: 800,
-    animation: {opacity:'show'},
+    animation: {opacity: 'show'},
     speed: 'fast',
     autoArrows: true,
     dropShadows: true,
@@ -101,8 +102,8 @@
   };
   $.fn.extend({
     hideSuperfishUl : function(){
-      var o = sf.op,
-        not = (o.retainPath===true) ? o.$path : '';
+      var o = sf.op;
+      var not = (o.retainPath === true) ? o.$path : '';
       o.retainPath = false;
       var $ul = $(['li.',o.hoverClass].join(''),this).add(this).not(not).removeClass(o.hoverClass)
           .children('ul').addClass('sf-hidden');
@@ -111,11 +112,10 @@
     },
     showSuperfishUl : function(){
       var o = sf.op,
-        sh = sf.c.shadowClass+'-off',
-        $ul = this.addClass(o.hoverClass)
-          .children('ul.sf-hidden').hide().removeClass('sf-hidden');
+        sh = sf.c.shadowClass + '-off',
+        $ul = this.addClass(o.hoverClass).children('ul.sf-hidden').hide().removeClass('sf-hidden');
       o.onBeforeShow.call($ul);
-      $ul.animate(o.animation,o.speed,function(){ o.onShow.call($ul); });
+      $ul.animate(o.animation, o.speed, function(){ o.onShow.call($ul); });
       return this;
     }
   });

supersubs.js

@@ -35,7 +35,7 @@
       if ($ULs.length) {
         // get the font size of menu.
         // .css('fontSize') returns various results cross-browser, so measure an em dash instead
-        var fontsize = $('<li id="menu-fontsize">&#8212;</li>'),
+        var fontsize = $('<li id="menu-fontsize">&#8212;</li>');
         var size = fontsize.attr('style','padding:0;position:absolute;top:-99999em;width:auto;')
         .appendTo($$)[0].clientWidth; //clientWidth is faster than width()
         // remove em dash