Skip to content
New issue

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

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

removing and then adding same tag didn't work #19

Closed
wants to merge 1 commit into from
Closed

removing and then adding same tag didn't work #19

wants to merge 1 commit into from

Conversation

loilo
Copy link

@loilo loilo commented Apr 28, 2014

removing tags only removed the related DOM element.
now also iterates over the "tags" array and removes entry there since this array is used to detect duplicates.
don't forget to change minified version. :)

removing tags only removed the related DOM element.
now also iterates over the "tags" array and removes entry there since this array is used to detect duplicates.
don't forget to change minified version. :)
@sniperwolf sniperwolf added the bug label Apr 28, 2014
@sniperwolf
Copy link
Owner

Hi @loilo,

thank you for this pull request. I just release the v1.3.1, where I refactored a little bit the code and enhanced the plugin with methods, that help you to work with taggingJS using JavaScript directly. I think that now is more flexible and also extensible. This version also include something that is similar to your changes. Look at remove method in tagging.js.

@sniperwolf sniperwolf closed this Apr 28, 2014
@loilo
Copy link
Author

loilo commented Apr 28, 2014

Nice stuff. Thought about things like that today when I used the plugin. Great plans, I'm excited to see the outcomes. :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants