Don't remove active button classes - screws up lots - closes #50 and #54

jtsage · Jun 11, 2012 · 6faf4e8 · 6faf4e8
1 parent 3b70062
commit 6faf4e8
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 10 deletions.
diff --git a/js/jquery.mobile.simpledialog2.js b/js/jquery.mobile.simpledialog2.js
@@ -313,8 +313,6 @@
 			if ( self.options.resizeListener === true ) { $(window).unbind('resize.simpledialog'); }
 		}
 
-		$.mobile.activePage.find('.ui-btn-active').removeClass('ui-btn-active');
-
 		if ( self.isDialog === true || self.options.animate === true ) {
 			setTimeout("$.mobile.sdCurrentDialog.destroy();", 1000);
 		} else {

diff --git a/tests/2dev.html b/tests/2dev.html
@@ -43,11 +43,12 @@ <h2>Fullscreen Mode</h2>
 			<p>The world looks mighty good to me, cause Tootsie Rolls are all I see. Whatever it is I think I see, becomes a Tootsie Roll to me! Tootsie Roll how I love your chocolatey chew, Tootsie Roll I think I'm in love with you. Whatever it is I think I see, becomes a Tootsie Roll to me!</p>
 			<p>You clicked: <span id="simpleboolout"></span></p>
 			<script type="text/javascript"> 
+				x = function () { console.log(this); }
 
 				$(document).delegate('#simplebool', 'click', function() {
 					$('<div>').simpledialog2({
-						mode : 'blank',
+						mode : 'button',
-
+						//themeDialog: 'd',
 						headerText: 'Bullshit',
 						headerClose: true,
 
@@ -58,15 +59,16 @@ <h2>Fullscreen Mode</h2>
 						'buttons' : {
 							'ok': {
 								'args' : ['something','else'],
+								'icon' : false,
 								'click': function (e) {
-									console.log(this.buttonPromptText.text());
+									//console.log(this.buttonPromptText.text());
-									console.log(arguments);
+									//console.log(arguments);
-									console.log(e);
+									//console.log(e);
 									return true;
 								}
 							},
 							'cancel': {
-								'click': function () { return true; }
+								'click': x 
 							}
 						},
 
@@ -79,8 +81,8 @@ <h2>Fullscreen Mode</h2>
 						//fullScreenForce: true,
 						blankContent : 
 							"<ul data-role='listview'><li>Some</li><li>List</li><li>Items</li></ul><a rel='close' data-role='button' href='#'>Close</a>",
-						callbackOpenArgs: ['some', 'shit'],
+						//callbackOpenArgs: ['some', 'shit'],
-						callbackOpen: function () { console.log(this); console.log(arguments); },
+						callbackClose: function () { console.log(this); console.log(arguments); },
 
 						})