Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Synced with https://github.com/vdepizzol/jquery-tokeninput #282

Open
wants to merge 3 commits into
from

Conversation

Projects
None yet
3 participants

and some fixes

Owner

loopj commented Feb 29, 2012

Thanks for the pull request.

I've been testing your version and it doesn't seem to work anymore. Clicking "Submit" on any of the examples in demo.html shows an alert box with "undefined", meaning that the value of the original input has no value.

Did you test these changes would not break existing functionality? Let me know if you think I'm doing something wrong.

Thanks

Adam Kusmierz Issue #282 bf470f4

First of all you have to use jquery with version minimum 1.6.0, as you use "prop" attribute (which was added in 1.6.0). The "prop" is used by your version, so nothing new here ;) but your demo.html had 1.5.x

Secondly, the vdepizzol's fork hides oryginal input by changing its type to hidden. So your selector from demo ("input[type=text]") can't work as there isn't any type=text field (only hidden). I changed this behavior and revert to yours.

The last but not least I made some mistakes ;) Additionaly I found that zipball (https://github.com/loopj/jquery-tokeninput/zipball/jquery-tokeninput-1.6.0) on your homepage is a little bit outdated, so I merged your changes too.

it's very simple ....just open your jquery.tokeninput.js file and replace the preventDuplicates: false to preventDuplicates: true ...

the line of this code is "34"

cheers.....
subbu vemuri

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment