From 2585ee58809c22a3c97f87b1b4aa68dbd41f386c Mon Sep 17 00:00:00 2001 From: "S. Andrew Sheppard" Date: Thu, 9 Jan 2014 15:27:18 -0600 Subject: [PATCH] AMD: Note on AMD best practices Fixes #14687 Closes gh-1478 --- AUTHORS.txt | 1 + src/exports/amd.js | 6 ++++++ 2 files changed, 7 insertions(+) diff --git a/AUTHORS.txt b/AUTHORS.txt index f5c06a6d2..b30e719a9 100644 --- a/AUTHORS.txt +++ b/AUTHORS.txt @@ -215,3 +215,4 @@ John Paul Jakob Stoeck Christopher Jones Forbes Lindesay +S. Andrew Sheppard diff --git a/src/exports/amd.js b/src/exports/amd.js index 815b5d88a..9a9846f9f 100644 --- a/src/exports/amd.js +++ b/src/exports/amd.js @@ -9,6 +9,12 @@ define([ // derived from file names, and jQuery is normally delivered in a lowercase // file name. Do this after creating the global so that if an AMD module wants // to call noConflict to hide this version of jQuery, it will work. + +// Note that for maximum portability, libraries that are not jQuery should +// declare themselves as anonymous modules, and avoid setting a global if an +// AMD loader is present. jQuery is a special case. For more information, see +// https://github.com/jrburke/requirejs/wiki/Updating-existing-libraries#wiki-anon + if ( typeof define === "function" && define.amd ) { define( "jquery", [], function() { return jQuery;