Skip to content
This repository was archived by the owner on Apr 12, 2024. It is now read-only.

Adding ng-jq functionality, the ability to choose your angular.element l... #10750

Closed
wants to merge 1 commit into from

Conversation

mboudreau
Copy link
Contributor

...ibrary or force jqLite even if jquery is available.

…t library or force jqLite even if jquery is available.
@@ -1147,7 +1177,7 @@ function snake_case(name, separator){

function bindJQuery() {
// bind to jQuery if present;
jQuery = window.jQuery;
jQuery = jq() ? window[jq()] : window.jQuery;
Copy link
Contributor

Choose a reason for hiding this comment

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

I think this will not do what you are expecting as '' is falsy

Copy link
Contributor

Choose a reason for hiding this comment

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

BTW, please do not call jq() twice

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@lgalfaso hm, guess I need to change my unit tests then. Thanks for the heads up. Will change right now.

@mboudreau mboudreau closed this Jan 15, 2015
@mboudreau
Copy link
Contributor Author

@lgalfaso

Added new pull request: #10761

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

Successfully merging this pull request may close these issues.

4 participants