search

drewwilson / AutoSuggest

jQuery AutoComplete Plugin
http://code.drewwilson.com/entry/autosuggest-jquery-plugin
86 stars 46 forks source link

Removing first element does not correctly update field values #11

Open Dachande663 opened 13 years ago

Dachande663 commented 13 years ago

jquery.autosuggest.js:168 values_input.val(values_input.val().replace(","+last+",",","));

Fails to replace the item if the item is the first in the series as there is no preceeding comma.

csanz commented 13 years ago

this is a big bug... it looks like autoSuggest maintenance is pretty slow.

Dachande663 commented 13 years ago

Fixed this in my local version by initially filling the field with a single comma, will submit a pull request when I get time.

csanz commented 13 years ago

Thanks man

Dachande663 commented 13 years ago

I haven't submitted a pull request yet, but the updated file I mentioned can be found here: https://github.com/Dachande663/Plex-Export/blob/v2/templates/bacon/assets/js/jquery.autoSuggest.js

csanz commented 13 years ago

Ok I made the update, thanks man!! send a pull request... I don't think he is actively maintaining this.... we should fork it and maintain a separate copy ha