ehynds / jquery-ui-multiselect-widget

jQuery UI MultiSelect widget
GNU General Public License v2.0
1.37k stars 692 forks source link

Option filter selects next options instead of the right ones... #374

Closed tvld closed 7 years ago

tvld commented 11 years ago

Unknown to this plugin, but was playing around and found that filer would select the next in line instead of the ones to be filtered! So for example, when entering "Option 3", it would show up with "Option 4"!

To get it working I had to change in jquery.multiselect.filter.js line 119 *120 to this :

rows.eq(i-1).show(); return inputs.get(i-1);

This must have been problem for more people?

ehynds commented 11 years ago

I can't reproduce this. What does your <select> markup look like?

tvld commented 11 years ago
<select id="lor_group_menu" name="example-optgroup" multiple="single" size="3">
    <optgroup label="My Groups">
        <option value="option1">V&amp;VN</option>
        <option value="option2">Lorinthe Development</option>
        <option value="option3">Management</option>
    </optgroup>
    <optgroup label="Recommended Groups">
        <option value="option4">Reumatologie</option>
        <option value="option5">Oncologie</option>
        <option value="option6">Dermatology</option>
        <option value="option7">Pharmacy</option>
    </optgroup>
     <optgroup label="Populair groups">
        <option value="option8">Nurses Employment</option>
    </optgroup>
      <optgroup label="Other groups">
        <option value="option12">Option 8</option>
        <option value="option13">Option 9</option>
        <option value="option14">Option 10</option>
        <option value="option15">Option 11</option>
    </optgroup>
</select>

And the js:

 $(function(){
            $("#lor_group_menu").multiselect({
                multiple: false,                
                noneSelectedText: "Select a Group"               
            }).multiselectfilter();
        });
ehynds commented 11 years ago

I still can't reproduce: http://jsfiddle.net/ehynds/wJCp8/1/

Also, multiple="single" will still enable multiple selection on the select box. You'll want to remove this attr altogether.

tvld commented 11 years ago

That's strange, is seems. Cant get my finger behind it with the time contraints today ( We decided to use jqueryui. autocomplete for now, but we will dig into this again in a few weeks as we this one will come back in other part of the code)

Fiddle fork here: http://jsfiddle.net/AjTxL/1/

We use jquery-1.8.3.js / jquery-ui-1.9.2.js