Skip to content

Commit 53099d8

Browse files
committed
Re-addresses jquery-archive#2776, and corrects an issue introduced in pull request jquery-archive#2783. Incorrect default theme was being set on the backdrop of dialog-style select menus.
1 parent 55d8989 commit 53099d8

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

js/jquery.mobile.dialog.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@
99
$.widget( "mobile.dialog", $.mobile.widget, {
1010
options: {
1111
closeBtnText : "Close",
12-
dialogTheme : "a",
12+
theme : "a",
1313
initSelector : ":jqmData(role='dialog')"
1414
},
1515
_create: function() {
@@ -22,7 +22,7 @@ $.widget( "mobile.dialog", $.mobile.widget, {
2222
$el.removeClass( pageTheme[ 0 ] );
2323
}
2424

25-
$el.addClass( "ui-body-" + this.options.dialogTheme );
25+
$el.addClass( "ui-body-" + this.options.theme );
2626

2727
// Class the markup for dialog styling
2828
// Set aria role

js/jquery.mobile.forms.select.custom.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@
1717
isMultiple = widget.isMultiple = widget.select[ 0 ].multiple,
1818
buttonId = selectID + "-button",
1919
menuId = selectID + "-menu",
20-
menuPage = $( "<div data-" + $.mobile.ns + "role='dialog' data-" +$.mobile.ns + "theme='"+ widget.options.menuPageTheme +"'>" +
20+
menuPage = $( "<div data-" + $.mobile.ns + "role='dialog' data-" +$.mobile.ns + "theme='"+ widget.options.overlayTheme +"'>" +
2121
"<div data-" + $.mobile.ns + "role='header'>" +
2222
"<div class='ui-title'>" + label.getEncodedText() + "</div>"+
2323
"</div>"+

0 commit comments

Comments
 (0)