fix to issue #91 - optgroup selected ignored

pianomansam · pianomansam · commit aa8719ea9834 · 2015-07-08T13:26:18.000-04:00
diff --git a/public/jquery.selectric.js b/public/jquery.selectric.js
@@ -9,7 +9,7 @@
  *    /,'
  *   /'
  *
- * Selectric Ϟ v1.9.3 (Jun 03 2015) - http://lcdsantos.github.io/jQuery-Selectric/
+ * Selectric Ϟ v1.9.3 (Jul 08 2015) - http://lcdsantos.github.io/jQuery-Selectric/
  *
  * Copyright (c) 2015 Leonardo Santos; Dual licensed: MIT/GPL
  *
@@ -178,7 +178,8 @@
 
           var $options = $original.children(),
               _$li = '<ul>',
-              selectedIndex = $options.filter(':selected').index(),
+              $justOptions = $original.find('option'),
+              selectedIndex = $justOptions.index($justOptions.filter(':selected')),
               currIndex = 0;
 
           currValue = (selected = ~selectedIndex ? selectedIndex : 0);
diff --git a/public/jquery.selectric.min.js b/public/jquery.selectric.min.js
diff --git a/src/jquery.selectric.js b/src/jquery.selectric.js
@@ -161,7 +161,8 @@
 
           var $options = $original.children(),
               _$li = '<ul>',
-              selectedIndex = $options.filter(':selected').index(),
+              $justOptions = $original.find('option'),
+              selectedIndex = $justOptions.index($justOptions.filter(':selected')),
               currIndex = 0;
 
           currValue = (selected = ~selectedIndex ? selectedIndex : 0);

Original file line number	Diff line number	Diff line change
`@@ -9,7 +9,7 @@`
`9`	`9`	`* /,'`
`10`	`10`	`* /'`
`11`	`11`	`*`
`12`		`- * Selectric Ϟ v1.9.3 (Jun 03 2015) - http://lcdsantos.github.io/jQuery-Selectric/`
	`12`	`+ * Selectric Ϟ v1.9.3 (Jul 08 2015) - http://lcdsantos.github.io/jQuery-Selectric/`
`13`	`13`	`*`
`14`	`14`	`* Copyright (c) 2015 Leonardo Santos; Dual licensed: MIT/GPL`
`15`	`15`	`*`
`@@ -178,7 +178,8 @@`
`178`	`178`
`179`	`179`	`var $options = $original.children(),`
`180`	`180`	`_$li = '<ul>',`
`181`		`- selectedIndex = $options.filter(':selected').index(),`
	`181`	`+ $justOptions = $original.find('option'),`
	`182`	`+ selectedIndex = $justOptions.index($justOptions.filter(':selected')),`
`182`	`183`	`currIndex = 0;`
`183`	`184`
`184`	`185`	`currValue = (selected = ~selectedIndex ? selectedIndex : 0);`