fixed: problem when hiding first option by css, thx to @RobertSheaO, see fnagel#152

fnagel · fnagel · commit edee0dcba2dc · 2011-08-09T22:01:47.000+02:00
diff --git a/ui/jquery.ui.selectmenu.js b/ui/jquery.ui.selectmenu.js
@@ -550,7 +550,7 @@ $.widget("ui.selectmenu", {
 			if ( $.browser.msie && $.browser.version.substr( 0,1 ) == 7 ) {
 				self._refreshPosition();
 			}
-			var selected = self.list.attr('aria-hidden', false).find('li:not(.' + self.widgetBaseClass + '-group):eq(' + self._selectedIndex() + ') a');
+			var selected = self.list.attr('aria-hidden', false).find('li:not(.' + self.widgetBaseClass + '-group):eq(' + self._selectedIndex() + '):visible a');
 			if (selected.length) selected[0].focus();
 			// positioning needed for FF, Chrome, IE8, IE7, IE6 (tested 01.08.11 on MS VPC Image)
 			self._refreshPosition();			

Original file line number	Diff line number	Diff line change
`@@ -550,7 +550,7 @@ $.widget("ui.selectmenu", {`
`550`	`550`	`if ( $.browser.msie && $.browser.version.substr( 0,1 ) == 7 ) {`
`551`	`551`	`self._refreshPosition();`
`552`	`552`	`}`
`553`		`- var selected = self.list.attr('aria-hidden', false).find('li:not(.' + self.widgetBaseClass + '-group):eq(' + self._selectedIndex() + ') a');`
	`553`	`+ var selected = self.list.attr('aria-hidden', false).find('li:not(.' + self.widgetBaseClass + '-group):eq(' + self._selectedIndex() + '):visible a');`
`554`	`554`	`if (selected.length) selected[0].focus();`
`555`	`555`	`// positioning needed for FF, Chrome, IE8, IE7, IE6 (tested 01.08.11 on MS VPC Image)`
`556`	`556`	`self._refreshPosition();`