From 64076ecfb5870130d89733ff34668fdd18683f9b Mon Sep 17 00:00:00 2001
From: Alexander Schmitz <arschmitz@gmail.com>
Date: Thu, 16 Jul 2015 09:14:58 -0400
Subject: [PATCH] Core: Move form method into its own module

Ref #9647
---
 tests/unit/core/core.js |  3 ++-
 ui/core.js              |  8 +-------
 ui/form-reset-mixin.js  |  2 +-
 ui/form.js              | 20 ++++++++++++++++++++
 4 files changed, 24 insertions(+), 9 deletions(-)
 create mode 100644 ui/form.js

diff --git a/tests/unit/core/core.js b/tests/unit/core/core.js
index ac9f28e00..0d25e22d8 100644
--- a/tests/unit/core/core.js
+++ b/tests/unit/core/core.js
@@ -1,7 +1,8 @@
 define( [
 	"jquery",
 	"lib/common",
-	"ui/core"
+	"ui/core",
+	"ui/form"
 ], function( $, common ) {
 
 module( "core - jQuery extensions" );
diff --git a/ui/core.js b/ui/core.js
index fe627dea6..fc91c2742 100644
--- a/ui/core.js
+++ b/ui/core.js
@@ -24,6 +24,7 @@
 			"./disable-selection",
 			"./escape-selector",
 			"./focusable",
+			"./form",
 			"./version"
 		], factory );
 	} 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() {
 		var ancestor, selector, id, labels, ancestors;
 
diff --git a/ui/form-reset-mixin.js b/ui/form-reset-mixin.js
index 423a35538..21704708e 100644
--- a/ui/form-reset-mixin.js
+++ b/ui/form-reset-mixin.js
@@ -4,7 +4,7 @@
 		// AMD. Register as an anonymous module.
 		define( [
 			"jquery",
-			"ui/core",
+			"./form",
 			"./version"
 		], factory );
 	} else {
diff --git a/ui/form.js b/ui/form.js
new file mode 100644
index 000000000..cfb4bd280
--- /dev/null
+++ b/ui/form.js
@@ -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 );
+};
+
+} ) );