Core: Move form method into its own module

Ref #9647
This commit is contained in:
Alexander Schmitz 2015-07-16 09:14:58 -04:00
parent 47188b7926
commit 64076ecfb5
4 changed files with 24 additions and 9 deletions

View File

@ -1,7 +1,8 @@
define( [ define( [
"jquery", "jquery",
"lib/common", "lib/common",
"ui/core" "ui/core",
"ui/form"
], function( $, common ) { ], function( $, common ) {
module( "core - jQuery extensions" ); module( "core - jQuery extensions" );

View File

@ -24,6 +24,7 @@
"./disable-selection", "./disable-selection",
"./escape-selector", "./escape-selector",
"./focusable", "./focusable",
"./form",
"./version" "./version"
], factory ); ], factory );
} else { } else {
@ -131,13 +132,6 @@ $.fn.extend( {
} ); } );
}, },
// Support: IE8 Only
// IE8 does not support the form attribute and when it is supplied. It overwrites the form prop
// with a string, so we need to find the proper form.
form: function() {
return typeof this[ 0 ].form === "string" ? this.closest( "form" ) : $( this[ 0 ].form );
},
labels: function() { labels: function() {
var ancestor, selector, id, labels, ancestors; var ancestor, selector, id, labels, ancestors;

View File

@ -4,7 +4,7 @@
// AMD. Register as an anonymous module. // AMD. Register as an anonymous module.
define( [ define( [
"jquery", "jquery",
"ui/core", "./form",
"./version" "./version"
], factory ); ], factory );
} else { } else {

20
ui/form.js Normal file
View File

@ -0,0 +1,20 @@
( function( factory ) {
if ( typeof define === "function" && define.amd ) {
// AMD. Register as an anonymous module.
define( [ "jquery", "./version" ], factory );
} else {
// Browser globals
factory( jQuery );
}
} ( function( $ ) {
// Support: IE8 Only
// IE8 does not support the form attribute and when it is supplied. It overwrites the form prop
// with a string, so we need to find the proper form.
return $.fn.form = function() {
return typeof this[ 0 ].form === "string" ? this.closest( "form" ) : $( this[ 0 ].form );
};
} ) );