From 62e163135300865f40d9172144d37ab35459e8f1 Mon Sep 17 00:00:00 2001 From: John Bender Date: Tue, 9 Aug 2011 15:50:42 -0700 Subject: [PATCH] fix label class application bug --- js/jquery.mobile.forms.select.custom.js | 4 ++-- js/jquery.mobile.forms.select.js | 2 ++ 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/js/jquery.mobile.forms.select.custom.js b/js/jquery.mobile.forms.select.custom.js index 0f5bab3b..74b1d283 100644 --- a/js/jquery.mobile.forms.select.custom.js +++ b/js/jquery.mobile.forms.select.custom.js @@ -395,8 +395,8 @@ var extensionAttributes = function(){ var widget = this, select = this.select, - selectID = this.select.attr( "id" ), - label = $( "label[for='"+ selectID +"']" ).addClass( "ui-select" ), + selectID = this.selectID, + label = this.label, thisPage = this.select.closest( ".ui-page" ), screen = $( "
", {"class": "ui-selectmenu-screen ui-screen-hidden"} ).appendTo( thisPage ), selectOptions = this.select.find("option"), diff --git a/js/jquery.mobile.forms.select.js b/js/jquery.mobile.forms.select.js index ce7c464c..e395d6ea 100644 --- a/js/jquery.mobile.forms.select.js +++ b/js/jquery.mobile.forms.select.js @@ -79,6 +79,8 @@ $.widget( "mobile.selectmenu", $.mobile.widget, { // setup items that are generally necessary for select menu extension _preExtension: function(){ this.select = this.element.wrap( "
" ); + this.selectID = this.select.attr( "id" ); + this.label = $( "label[for='"+ this.selectID +"']" ).addClass( "ui-select" ); this.isMultiple = this.select[ 0 ].multiple; this.options.theme = this._theme(); this.selectOptions = this.select.find( "option" );