From 8fb64a5ef91b35f1d1266c005ed4075d41983b74 Mon Sep 17 00:00:00 2001
From: Konstantin Pozin <kpozin@gmail.com>
Date: Mon, 18 Jul 2011 09:27:39 -0700
Subject: [PATCH] Added cached <option> element indices to corresponding <li>s
 in custom select menus. Removed redundant .eq() call.

---
 js/jquery.mobile.forms.select.js | 14 ++++++++------
 1 file changed, 8 insertions(+), 6 deletions(-)

diff --git a/js/jquery.mobile.forms.select.js b/js/jquery.mobile.forms.select.js
index 5010cb00..1973951e 100644
--- a/js/jquery.mobile.forms.select.js
+++ b/js/jquery.mobile.forms.select.js
@@ -229,17 +229,19 @@ $.widget( "mobile.selectmenu", $.mobile.widget, {
 				})
 				.delegate( "li:not(.ui-disabled, .ui-li-divider)", "vclick", function( event ) {
 
-					// index of option tag to be selected
-					var oldIndex = select[ 0 ].selectedIndex,
-						newIndex = list.find( "li:not(.ui-li-divider)" ).index( this ),
-						option = self.optionElems.eq( newIndex )[ 0 ];
+					
+					var $this = $( this ),
+						// index of option tag to be selected
+						oldIndex = select[ 0 ].selectedIndex,
+						newIndex = $this.jqmData("option-index"),
+						option = self.optionElems[ newIndex ];
 
 					// toggle selected status on the tag for multi selects
 					option.selected = isMultiple ? !option.selected : true;
 
 					// toggle checkbox class for multiple selects
 					if ( isMultiple ) {
-						$( this ).find( ".ui-icon" )
+						$this.find( ".ui-icon" )
 							.toggleClass( "ui-icon-checkbox-on", option.selected )
 							.toggleClass( "ui-icon-checkbox-off", !option.selected );
 					}
@@ -373,7 +375,7 @@ $.widget( "mobile.selectmenu", $.mobile.widget, {
 				extraAttrs.push( "aria-disabled='true'" );
 			}
 
-			lis.push( "<li data-" + $.mobile.ns + "icon='"+ dataIcon +"' class='"+ classes.join(" ") + "' " + extraAttrs.join(" ") +">"+ anchor +"</li>" )
+			lis.push( "<li data-" + $.mobile.ns + "option-index='" + i + "' data-" + $.mobile.ns + "icon='"+ dataIcon +"' class='"+ classes.join(" ") + "' " + extraAttrs.join(" ") +">"+ anchor +"</li>" )
 		});
 
 		self.list.html( lis.join(" ") );