Skip to content
This repository has been archived by the owner on Feb 29, 2024. It is now read-only.

Adding support for loading #57

Closed
Krinkle opened this issue May 23, 2014 · 1 comment
Closed

Adding support for loading #57

Krinkle opened this issue May 23, 2014 · 1 comment

Comments

@Krinkle
Copy link
Owner

Krinkle commented May 23, 2014

From tim@timtucker.com on February 24, 2012 17:12:43

Our of curiosity, have you considered adding in support for loading jquery-json as an AMD module?

Given that jQuery core added in support for AMD-based module loaders recently, it would seem like it might prove useful to a number of people.

More details on some of the methods for supporting AMD: https://github.com/umdjs/umd As an example, all it should really take is using:

(function (factory) {
    if (typeof define === 'function' && define.amd) {
        // AMD. Register as an anonymous module.
        define(['jquery'], factory);
    } else {
        // Browser globals
        factory(jQuery);
    }
}
(function( $ ) {
...
plugin code
...
}));

instead of:

(function( $ ) {
...
plugin code
...
})( jQuery );

Original issue: http://code.google.com/p/jquery-json/issues/detail?id=57

@Krinkle
Copy link
Owner Author

Krinkle commented May 23, 2014

From krinklemail on November 13, 2012 17:42:28

Labels: -Type-Defect Type-Enhancement

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

No branches or pull requests

1 participant