Commit cecb52ff authored by Timmy Willison's avatar Timmy Willison

Specify support as a dependency wherever it is used. Optimize module order to save 15 bytes.

parent 7877c4fa
...@@ -86,7 +86,7 @@ Some example modules that can be excluded are: ...@@ -86,7 +86,7 @@ Some example modules that can be excluded are:
- **offset**: The `.offset()`, `.position()`, `.offsetParent()`, `.scrollLeft()`, and `.scrollTop()` methods. - **offset**: The `.offset()`, `.position()`, `.offsetParent()`, `.scrollLeft()`, and `.scrollTop()` methods.
- **wrap**: The `.wrap()`, `.wrapAll()`, `.wrapInner()`, and `.unwrap()` methods. - **wrap**: The `.wrap()`, `.wrapAll()`, `.wrapInner()`, and `.unwrap()` methods.
- **exports/amd**: Exclude the AMD definition. - **exports/amd**: Exclude the AMD definition.
- **core/ready**: Exclude the ready module if you place your scripts at the end of the body. Any ready callbacks will simply be called immediately. - **core/ready**: Exclude the ready module if you place your scripts at the end of the body. Any ready callbacks bound with `jQuery()` will simply be called immediately. However, `jQuery(document).ready()` will not be a function and `.on("ready", ...)` or similar will not be triggered.
- **deferred**: Exclude jQuery.Deferred. This also removes jQuery.Callbacks. *Note* that modules that depend on jQuery.Deferred(AJAX, effects, core/ready) will not be removed and will still expect jQuery.Deferred to be there. Include your own jQuery.Deferred implementation or exclude those modules as well (`grunt custom:-deferred,-ajax,-effects,-core/ready`). - **deferred**: Exclude jQuery.Deferred. This also removes jQuery.Callbacks. *Note* that modules that depend on jQuery.Deferred(AJAX, effects, core/ready) will not be removed and will still expect jQuery.Deferred to be there. Include your own jQuery.Deferred implementation or exclude those modules as well (`grunt custom:-deferred,-ajax,-effects,-core/ready`).
- **support**: Excluding the support module is not recommended, but possible. It's your responsibility to either remove modules that use jQuery.support (many of them) or replace the values wherever jQuery.support is used. This is mainly only useful when building a barebones version of jQuery. - **support**: Excluding the support module is not recommended, but possible. It's your responsibility to either remove modules that use jQuery.support (many of them) or replace the values wherever jQuery.support is used. This is mainly only useful when building a barebones version of jQuery.
......
define([ define([
"../core", "../core",
"../ajax" "../ajax",
"../support"
], function( jQuery ) { ], function( jQuery ) {
jQuery.ajaxSettings.xhr = function() { jQuery.ajaxSettings.xhr = function() {
......
...@@ -2,7 +2,8 @@ define([ ...@@ -2,7 +2,8 @@ define([
"../core", "../core",
"../var/rnotwhite", "../var/rnotwhite",
"../var/strundefined", "../var/strundefined",
"../selector" "../selector",
"../support"
], function( jQuery, rnotwhite, strundefined ) { ], function( jQuery, rnotwhite, strundefined ) {
var nodeHook, boolHook; var nodeHook, boolHook;
......
define([ define([
"../core" "../core",
"../support"
], function( jQuery ) { ], function( jQuery ) {
var rfocusable = /^(?:input|select|textarea|button)$/i; var rfocusable = /^(?:input|select|textarea|button)$/i;
......
define([ define([
"../core" "../core",
"../support"
], function( jQuery ) { ], function( jQuery ) {
var rreturn = /\r/g; var rreturn = /\r/g;
......
...@@ -6,7 +6,9 @@ define([ ...@@ -6,7 +6,9 @@ define([
"./css/defaultDisplay", "./css/defaultDisplay",
"./data/var/data_priv", "./data/var/data_priv",
"./core/swap", "./core/swap",
"./selector" // contains "./core/ready",
"./selector", // contains
"./support"
], function( jQuery, pnum, cssExpand, isHidden, defaultDisplay, data_priv ) { ], function( jQuery, pnum, cssExpand, isHidden, defaultDisplay, data_priv ) {
var curCSS, var curCSS,
// swappable if display is none or starts with table except "table", "table-cell", or "table-caption" // swappable if display is none or starts with table except "table", "table-cell", or "table-caption"
......
define([ define([
"./core", "./core",
"./selector", "./selector",
"./traversing",
"./callbacks", "./callbacks",
"./deferred", "./deferred",
"./core/ready", "./core/ready",
"./traversing", "./support",
"./data", "./data",
"./queue", "./queue",
"./queue/delay", "./queue/delay",
...@@ -26,7 +27,6 @@ define([ ...@@ -26,7 +27,6 @@ define([
"./effects/animated-selector", "./effects/animated-selector",
"./offset", "./offset",
"./dimensions", "./dimensions",
"./support",
"./deprecated" "./deprecated"
], function( jQuery ) { ], function( jQuery ) {
......
...@@ -8,7 +8,8 @@ define([ ...@@ -8,7 +8,8 @@ define([
"./data/accepts", "./data/accepts",
"./selector", "./selector",
"./traversing", "./traversing",
"./event" "./event",
"./support"
], function( jQuery, concat, push, rcheckableType, data_priv, data_user ){ ], function( jQuery, concat, push, rcheckableType, data_priv, data_user ){
var rxhtmlTag = /<(?!area|br|col|embed|hr|img|input|link|meta|param)(([\w:]+)[^>]*)\/>/gi, var rxhtmlTag = /<(?!area|br|col|embed|hr|img|input|link|meta|param)(([\w:]+)[^>]*)\/>/gi,
......
...@@ -2,7 +2,7 @@ define([ ...@@ -2,7 +2,7 @@ define([
"./core", "./core",
"./var/strundefined", "./var/strundefined",
"./css", "./css",
"./selector" "./selector" // contains
], function( jQuery, strundefined ) { ], function( jQuery, strundefined ) {
var docElem = window.document.documentElement; var docElem = window.document.documentElement;
...@@ -136,7 +136,7 @@ jQuery.fn.extend({ ...@@ -136,7 +136,7 @@ jQuery.fn.extend({
return this.map(function() { return this.map(function() {
var offsetParent = this.offsetParent || docElem; var offsetParent = this.offsetParent || docElem;
while ( offsetParent && ( !jQuery.nodeName( offsetParent, "html" ) && jQuery.css( offsetParent, "position") === "static" ) ) { while ( offsetParent && ( !jQuery.nodeName( offsetParent, "html" ) && jQuery.css( offsetParent, "position" ) === "static" ) ) {
offsetParent = offsetParent.offsetParent; offsetParent = offsetParent.offsetParent;
} }
......
define([ define([
"./core", "./core",
"./core/swap" "./core/swap",
// This is listed as a dependency for build order, but it's still optional in builds
"./core/ready"
], function( jQuery ) { ], function( jQuery ) {
jQuery.support = (function( support ) { jQuery.support = (function( support ) {
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment