search

emposha / FCBKcomplete

Jquery facebook like(fancy) multi-select
http://www.emposha.com
383 stars 115 forks source link

2.7.5 version fails to load tags present in select #32

Closed fizyk closed 13 years ago

fizyk commented 13 years ago

After switching fcbkcomplete from 2.7.4 to 2.7.5, fcbkcomplete fails to load and render tags already present. I seem to have the same problem for the provided example index.html from fcbk repo for the latest 2.7.5 release

cweiske commented 13 years ago

Such problems can be avoided with the unit test suite that I started in my fork, branch "unittests".

emposha commented 13 years ago

I believe it`s because new key:value structure. just check new data file

fizyk commented 13 years ago

I don't think so. It works pretty well with old data feed structure. Problem is in reading those already present in select (example index.html doesn't show any initial tags, although there are three defined)

markitup commented 13 years ago

Hi, fcbkcomplete needs to have

To keep a unobtrusive behavior replace line 124: if (option.hasClass("selected")) { By: if (option.attr("selected")) { and it works.

Thanks you for your great work.

emposha commented 13 years ago

@markitup

if (option.attr("selected")) { this approach cause strange bugs with ie, try to search in google.