r2792 - replaced some hard coding with _selectedOptionLi()

r2792 - replaced some hard coding with _selectedOptionLi()


Author: scottjehl
Date: Tue Jun 16 14:05:32 2009
New Revision: 2792
Modified:
branches/labs/selectmenu/ui.selectmenu.js
Log:
replaced some hard coding with _selectedOptionLi()
Modified: branches/labs/selectmenu/ui.selectmenu.js
==============================================================================
--- branches/labs/selectmenu/ui.selectmenu.js    (original)
+++ branches/labs/selectmenu/ui.selectmenu.js    Tue Jun 16 14:05:32 2009
@@ -132,16 +132,16 @@
                    return false;
                })
                .bind('mouseover focus', function(){
-                    $(this).parent().find('.'+ self.widgetBaseClass
+ '-item-selected').addClass(activeClass);
-                    $(this).parent().find('.'+ self.widgetBaseClass
+ '-item-focus').removeClass(self.widgetBaseClass+'-item-focus
ui-state-hover');
+                    self._selectedOptionLi().addClass(activeClass);
+                    self._focusedOptionLi().removeClass(self.widgetBaseClass+'-item-focus
ui-state-hover');
                    $(this).removeClass('ui-state-active').addClass(self.widgetBaseClass
+ '-item-focus ui-state-hover');
                    //if it's a popup menu, if the hovered item isn't selected, let the
mouseup event fire
-                    if(o.style == "popup" && !$(this).is('.'+ self.widgetBaseClass
+ '-item-selected')){
+                    if(o.style == "popup" && !$(this).is(self._selectedOptionLi())){
                        self._safemouseup = true;
                    }    
                })
                .bind('mouseout blur', function(){
-                    if($(this).is('.'+ self.widgetBaseClass + '-item-selected')){
$(this).addClass(activeClass); }
+                    if($(this).is( self._selectedOptionLi() )){
$(this).addClass(activeClass); }
                    $(this).removeClass(self.widgetBaseClass + '-item-focus
ui-state-hover');
                });