Skip to content
This repository has been archived by the owner on Oct 2, 2019. It is now read-only.

Allow creating new tags in multiple select mode. #226

Closed
wants to merge 3 commits into from
Closed

Allow creating new tags in multiple select mode. #226

wants to merge 3 commits into from

Conversation

pderksen
Copy link
Contributor

In multiple select (tag) mode, when Enter key pressed and tag doesn't exist, add it.

See issue #179

@pderksen
Copy link
Contributor Author

Added commit to help separate issues #187 & #220.

@keelerm84
Copy link

I would love to see either this or #63 merged in.

@zavidovych
Copy link

Do you think you could add the following:
if (ctrl.multiple && ctrl.freeTags && !processed && ctrl.search && key == KEY.ENTER) {
then define freeTags as an attribute?

@j-walker23
Copy link

This is awesome, thanks for much for putting in the time to add it. Is there a timeframe for when it will be merged? Using the stupid ng-tags-input module for this and the ui-select is/will be so much nicer. Plus really cool to have all features in one library. You guys have kicked ass with ui-select!!

@j-walker23
Copy link

That is awesome! Thank you.

On Sun, Oct 5, 2014 at 5:01 PM, Thomas Vervest notifications@github.com
wrote:

Bad news: I really need this feature for a project I'm working on.
Good news: I'm going to make some time tomorrow to try and apply the patch
on the current master.
So in other words, with a bit of luck I'll have a new pull request ready
tomorrow which can be merged direct into the current master. I'll keep you
guys posted!


Reply to this email directly or view it on GitHub
#226 (comment).

@brianfeister brianfeister mentioned this pull request Oct 6, 2014
@brianfeister
Copy link

There are a number of similar commits for tagging support ... and I don't see any comments from the repo maintainer. Will any be merged? #226, #198, #260

@dimirc
Copy link
Contributor

dimirc commented Oct 10, 2014

Related to #260

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

Successfully merging this pull request may close these issues.

6 participants