Widgets: Remove mobile base widget and extend $.widget instead. Fixes #6132

arschmitz · arschmitz · commit 245e3d1d0e9f · 2013-07-03T23:14:53.000-04:00
diff --git a/demos/_assets/js/h2widget.js b/demos/_assets/js/h2widget.js
@@ -101,7 +101,7 @@
 	});
 	//h2 widget
 	$( document ).on( "mobileinit", function(){
-		$.widget( "mobile.h2linker", $.mobile.widget, {
+		$.widget( "mobile.h2linker", {
 			options:{
 				initSelector: ":jqmData(quicklinks='true')"
 			},
diff --git a/js/jquery.mobile.widget.js b/js/jquery.mobile.widget.js
@@ -8,7 +8,7 @@ define( [ "jquery", "./jquery.mobile.ns", "jquery.ui.widget" ], function( jQuery
 //>>excludeEnd("jqmBuildExclude");
 (function( $, undefined ) {
 
-$.widget( "mobile.widget", {
+$.extend( $.Widget.prototype, {
 	_getCreateOptions: function() {
 
 		var elem = this.element,
@@ -57,7 +57,7 @@ $.widget( "mobile.widget", {
 		throw "Widget [" + this.widgetName + "]: " + msg;
 	}
 });
-
+$.mobile.widget = $.Widget;
 })( jQuery );
 //>>excludeStart("jqmBuildExclude", pragmas.jqmBuildExclude);
 });
diff --git a/js/widgets/collapsible.js b/js/widgets/collapsible.js
@@ -15,7 +15,7 @@ define( [
 
 var getAttr = $.mobile.getAttribute;
 
-$.widget( "mobile.collapsible", $.mobile.widget, {
+$.widget( "mobile.collapsible", {
 	options: {
 		expandCueText: " click to expand contents",
 		collapseCueText: " click to collapse contents",
diff --git a/js/widgets/collapsibleSet.js b/js/widgets/collapsibleSet.js
@@ -9,7 +9,7 @@ define( [ "jquery", "../jquery.mobile.widget", "./collapsible", "./addFirstLastC
 //>>excludeEnd("jqmBuildExclude");
 (function( $, undefined ) {
 
-$.widget( "mobile.collapsibleset", $.mobile.widget, $.extend( {
+$.widget( "mobile.collapsibleset", $.extend( {
 	_create: function() {
 		var $el = this.element,
 			o = this.options,
diff --git a/js/widgets/controlgroup.js b/js/widgets/controlgroup.js
@@ -12,7 +12,7 @@ define( [ "jquery",
 //>>excludeEnd("jqmBuildExclude");
 (function( $, undefined ) {
 
-$.widget( "mobile.controlgroup", $.mobile.widget, $.extend( {
+$.widget( "mobile.controlgroup", $.extend( {
 	options: {
 		theme: null,
 		shadow: false,
diff --git a/js/widgets/dialog.js b/js/widgets/dialog.js
@@ -9,7 +9,7 @@ define( [ "jquery", "../jquery.mobile.widget", "./page", "../jquery.mobile.navig
 //>>excludeEnd("jqmBuildExclude");
 (function( $, window, undefined ) {
 
-$.widget( "mobile.dialog", $.mobile.widget, $.extend( {
+$.widget( "mobile.dialog", $.extend( {
 	options: {
 		closeBtn: "left", /* Accepts left, right and none */
 		closeBtnText: "Close",
diff --git a/js/widgets/forms/button.js b/js/widgets/forms/button.js
@@ -9,7 +9,7 @@ define( [ "jquery", "../../jquery.mobile.widget", "../../jquery.mobile.registry"
 //>>excludeEnd("jqmBuildExclude");
 (function( $, undefined ) {
 
-$.widget( "mobile.button", $.mobile.widget, {
+$.widget( "mobile.button", {
 	options: {
 		theme: null,
 		icon: null,
diff --git a/js/widgets/forms/checkboxradio.js b/js/widgets/forms/checkboxradio.js
@@ -17,7 +17,7 @@ define( [ "jquery",
 //>>excludeEnd("jqmBuildExclude");
 (function( $, undefined ) {
 
-$.widget( "mobile.checkboxradio", $.mobile.widget, $.extend( {
+$.widget( "mobile.checkboxradio", $.extend( {
 	options: {
 		theme: null,
 		mini: false
diff --git a/js/widgets/forms/rangeslider.js b/js/widgets/forms/rangeslider.js
@@ -14,7 +14,7 @@ define( [ "jquery",
 	"./slider" ], function( jQuery ) {
 //>>excludeEnd("jqmBuildExclude");
 (function( $, undefined ) {
-	$.widget( "mobile.rangeslider", $.mobile.widget, $.extend( {
+	$.widget( "mobile.rangeslider", $.extend( {
 
 		options: {
 			theme: null,
diff --git a/js/widgets/forms/select.js b/js/widgets/forms/select.js
@@ -9,7 +9,7 @@ define( [ "jquery", "../../jquery.mobile.core", "../../jquery.mobile.widget", ".
 //>>excludeEnd("jqmBuildExclude");
 (function( $, undefined ) {
 
-$.widget( "mobile.selectmenu", $.mobile.widget, $.extend( {
+$.widget( "mobile.selectmenu", $.extend( {
 	options: {
 		theme: null,
 		disabled: false,
diff --git a/js/widgets/forms/slider.js b/js/widgets/forms/slider.js
@@ -14,7 +14,7 @@ define( [ "jquery",
 //>>excludeEnd("jqmBuildExclude");
 (function( $, undefined ) {
 
-$.widget( "mobile.slider", $.mobile.widget, $.extend( {
+$.widget( "mobile.slider", $.extend( {
 	widgetEventPrefix: "slide",
 
 	options: {
diff --git a/js/widgets/forms/textinput.js b/js/widgets/forms/textinput.js
@@ -9,7 +9,7 @@ define( [ "jquery", "../../jquery.mobile.core", "../../jquery.mobile.widget", ".
 //>>excludeEnd("jqmBuildExclude");
 (function( $, undefined ) {
 
-$.widget( "mobile.textinput", $.mobile.widget, {
+$.widget( "mobile.textinput", {
 	options: {
 		theme: null,
 		corners: true,
diff --git a/js/widgets/listview.js b/js/widgets/listview.js
@@ -11,7 +11,7 @@ define( [ "jquery", "../jquery.mobile.widget", "./page", "./addFirstLastClasses"
 
 var getAttr = $.mobile.getAttribute;
 
-$.widget( "mobile.listview", $.mobile.widget, $.extend( {
+$.widget( "mobile.listview", $.extend( {
 
 	options: {
 		theme: null,
diff --git a/js/widgets/navbar.js b/js/widgets/navbar.js
@@ -10,7 +10,7 @@ define( [ "jquery", "../jquery.mobile.widget", "../jquery.mobile.grid", "../jque
 //>>excludeEnd("jqmBuildExclude");
 (function( $, undefined ) {
 
-$.widget( "mobile.navbar", $.mobile.widget, {
+$.widget( "mobile.navbar", {
 	options: {
 		iconpos: "top",
 		grid: null
diff --git a/js/widgets/page.js b/js/widgets/page.js
@@ -7,7 +7,7 @@ define( [ "jquery", "../jquery.mobile.widget", "../jquery.mobile.core", "../jque
 //>>excludeEnd("jqmBuildExclude");
 (function( $, undefined ) {
 
-$.widget( "mobile.page", $.mobile.widget, {
+$.widget( "mobile.page", {
 	options: {
 		theme: "a",
 		domCache: false,
@@ -55,7 +55,6 @@ $.widget( "mobile.page", $.mobile.widget, {
 	},
 
 	_setOptions: function( o ) {
-		
 		if( o.theme !== undefined ) {
 			this.element.removeClass( "ui-body-" + this.options.theme ).addClass( "ui-body-" + o.theme );
 		}
diff --git a/js/widgets/panel.js b/js/widgets/panel.js
@@ -9,7 +9,7 @@ define( [ "jquery", "../jquery.mobile.widget", "./page", "../jquery.mobile.regis
 //>>excludeEnd("jqmBuildExclude");
 (function( $, undefined ) {
 
-$.widget( "mobile.panel", $.mobile.widget, {
+$.widget( "mobile.panel", {
 	options: {
 		classes: {
 			panel: "ui-panel",
diff --git a/js/widgets/popup.js b/js/widgets/popup.js
@@ -52,7 +52,7 @@ function windowCoords() {
 	};
 }
 
-$.widget( "mobile.popup", $.mobile.widget, {
+$.widget( "mobile.popup", {
 	options: {
 		theme: null,
 		overlayTheme: null,
diff --git a/js/widgets/table.js b/js/widgets/table.js
@@ -9,7 +9,7 @@ define( [ "jquery", "../jquery.mobile.widget", "./page", "../jquery.mobile.regis
 //>>excludeEnd("jqmBuildExclude");
 (function( $, undefined ) {
 
-$.widget( "mobile.table", $.mobile.widget, {
+$.widget( "mobile.table", {
 	options: {
 		classes: {
 			table: "ui-table"
diff --git a/js/widgets/tabs.js b/js/widgets/tabs.js
@@ -8,11 +8,9 @@ define( [ "jquery", "../jquery.mobile.widget", "../jquery.mobile.core","./jquery
 //>>excludeEnd("jqmBuildExclude");
 (function( $, undefined ) {
 
-	$.widget("mobile.tabs", $.mobile.widget, $.ui.tabs.prototype);
+	$.ui.tabs.initSelector = ":jqmData(role='tabs'), :jqmData(content='tabs')";
 
-	$.mobile.tabs.initSelector = ":jqmData(role='tabs'), :jqmData(content='tabs')";
-
-	$.mobile._enhancer.add( "mobile.tabs" );
+	$.mobile._enhancer.add( "ui.tabs" );
 
 })( jQuery );
 //>>excludeStart("jqmBuildExclude", pragmas.jqmBuildExclude);
diff --git a/js/widgets/toolbar.js b/js/widgets/toolbar.js
@@ -10,7 +10,7 @@ define( [ "jquery", "../jquery.mobile.widget", "../jquery.mobile.core", "../jque
 (function( $, undefined ) {
 
 
-	$.widget( "mobile.toolbar", $.mobile.widget, {
+	$.widget( "mobile.toolbar", {
 		options: {
 			theme: null,
 			addBackBtn: false,
