Skip to content
This repository was archived by the owner on Sep 8, 2020. It is now read-only.

Update select2.js #106

Closed
wants to merge 1 commit into from
Closed

Update select2.js #106

wants to merge 1 commit into from

Conversation

jameskleeh
Copy link

In AngularJS 1.2.1 the newly defined controller.$render method never got called. I believe because the method already exists and the call is done above the new declaration. Could be wrong on that though - just a guess. I just changed the method name to be unique and it is called now.

In AngularJS 1.2.1 the newly defined controller.$render method never got called. I believe because the method already exists and the call is declared above the new declaration. Could be wrong on that though - just a guess. I just changed the method name to be unique and it is called now.
@shaunrader
Copy link

Appears to be related to #107 and can also be fixed by changing the select2's priority to something greater than 0. I found more info on the underlying issue of the directive priority order changing with angular > 1.2 here (originally found here)

@ProLoser
Copy link
Member

According to that blog post, the priority is the issue, not the render function.

@ProLoser ProLoser closed this in 6b20bd7 Nov 22, 2013
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.

3 participants