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

Pass jshint, better pasting, some small tweaks #9

Closed
wants to merge 19 commits into from

Conversation

mikemaccana
Copy link
Contributor

This looks like a bigger change than it is. Three parts:

  • pass jshint - add some semis, put if bodies on separate lines, etc. Hope that's OK, I use jshint a lot and it's nice to keep quiet so that when it complains you know it's important
  • when pasting, treat all individual values with the same steps - blank handling, duplicate handling, etc - that we were ding on individual inputs earlier
  • when pasting, make the tags immediately - before we had to add a single trailing , to build the tags.

Hope that's OK. I've bumped the ver in both files to 0.8.1.

// so that users who only want one thing don't have to enter commas
if ( ! $('.tag', true).length ) {
originalInput.value = getValue();
originalInput.dispatchEvent(new Event('originalInput'));
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hmm, this won't work. input is an actual event in browsers, originalInput is not.

@mikemaccana
Copy link
Contributor Author

Thanks @developit! Sorted the bad event, have adjusted PR according to comments.

developit added a commit that referenced this pull request Aug 23, 2015
Fixes #5 and #10, and brings in most of @mikemaccana's changes from merged PR #9.
@developit
Copy link
Owner

Manually merged ideas/chunks of this as part of #11. README update still needed though.

@developit developit closed this Aug 24, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants