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

pagerjs AMD support replies on global $ variable #135

Closed
3cp opened this issue Jul 19, 2013 · 1 comment
Closed

pagerjs AMD support replies on global $ variable #135

3cp opened this issue Jul 19, 2013 · 1 comment
Milestone

Comments

@3cp
Copy link

@3cp 3cp commented Jul 19, 2013

Hi Finnsson,
The pager.js amd implementation relies on jQuery lib registers itself as the global variable "$", which is true by default. But developer may choose to use jQuery noConflict to turn off global variables "$" and "jQuery" to conform with the idea of AMD.
http://requirejs.org/docs/jquery.html#noconflictmap

Should not this

define('pager', ['knockout', 'jquery'], function (ko) {
    return pagerJsModule($, ko);
});

be updated to

define('pager', ['knockout', 'jquery'], function (ko, $) { ... });

?

@finnsson finnsson closed this in 690386e Aug 1, 2013
@finnsson
Copy link
Owner

@finnsson finnsson commented Aug 1, 2013

Thanks for the bug report.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.