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

fix issue with tagging, when mouse click on options doesn't work for angular > 1.4.x #1335

Closed
wants to merge 1 commit into from

Conversation

vitalii
Copy link

@vitalii vitalii commented Nov 30, 2015

There is problem for tagging with ang > 1.4.x , and 1.3.x.
It's only working with tagging-label="false"
#983 (comment)
Here is descrobed workaround with tagging-label="false".
But at the same time doesn't work choosing by mouse.
In this pull request fix for it

@vitalii
Copy link
Author

vitalii commented Nov 30, 2015

seems, that error with The Travis CI build failed not related with pull request

Error: [$injector:nomod] Module 'ui.select' is not available! You either misspelled the module name or forgot to load it. If registering a module ensure that you specify the dependencies as the second argument.
http://errors.angularjs.org/1.4.8/$injector/nomod?p0=ui.select
at /home/travis/build/angular-ui/ui-select/bower_components/angular/angular.js:68

Few builds before have the same problem

@vitalii
Copy link
Author

vitalii commented Nov 30, 2015

tests locally

[15:09:49] Using gulpfile ~/projects/my-ui-select/gulpfile.js
[15:09:49] Starting 'clean'...
[15:09:49] Finished 'clean' after 8.47 ms
[15:09:49] Starting 'scripts'...
[15:09:49] Starting 'styles'...
[15:09:49] Finished 'styles' after 62 ms
[15:09:50] Finished 'scripts' after 1.2 s
[15:09:50] Starting 'build'...
[15:09:50] Finished 'build' after 8.16 μs
[15:09:50] Starting 'karma'...
[15:09:51] Finished 'karma' after 225 ms
[15:09:51] Starting 'test'...
[15:09:51] Finished 'test' after 2.36 μs
WARN [karma]: Port 9876 in use
INFO [karma]: Karma v0.12.37 server started at http://localhost:9877/
INFO [launcher]: Starting browser Chrome
INFO [Chrome 45.0.2454 (Mac OS X 10.10.4)]: Connected on socket Z87iWqaqHyTmgpjd8Vqq with id 17696757
Chrome 45.0.2454 (Mac OS X 10.10.4): Executed 134 of 134 SUCCESS (3.019 secs / 2.994 secs)

@@ -519,7 +519,7 @@ uis.controller('uiSelectCtrl',
if (!item || angular.equals( ctrl.items[0], item ) ) {
return;
}
} else {
} else if(!$event){ //if not mouse click, that done to prevent setting value by activeIndex
Copy link
Contributor

Choose a reason for hiding this comment

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

The README clearly states:

When issuing a pull request, please exclude changes from the "dist" folder to avoid merge conflicts.

@aaronroberson
Copy link
Contributor

@vitalii Can you confirm fixed and close this?

@user378230 user378230 closed this Nov 11, 2016
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.

4 participants