diff options
author | ddstreet <ddstreet@ieee.org> | 2011-03-11 10:52:13 -0500 |
---|---|---|
committer | Scott González <scott.gonzalez@gmail.com> | 2011-03-11 10:52:13 -0500 |
commit | 0b30a1d450f6b7d529732d6a39c5f4057efaaaf4 (patch) | |
tree | 67fb01a30db6900f21d9b179af8835078140b4fc /ui | |
parent | 0eb1106b5b7a1ad01d76c38b1cbb5b16e55ca156 (diff) | |
download | jquery-ui-0b30a1d450f6b7d529732d6a39c5f4057efaaaf4.tar.gz jquery-ui-0b30a1d450f6b7d529732d6a39c5f4057efaaaf4.zip |
Button: find associated label even without common ancestor. Fixes #7092 - button creation that requires a matching label does not find label in all cases
Diffstat (limited to 'ui')
-rw-r--r-- | ui/jquery.ui.button.js | 12 |
1 files changed, 10 insertions, 2 deletions
diff --git a/ui/jquery.ui.button.js b/ui/jquery.ui.button.js index 134dae3d3..9cf8974b2 100644 --- a/ui/jquery.ui.button.js +++ b/ui/jquery.ui.button.js @@ -201,8 +201,16 @@ $.widget( "ui.button", { if ( this.type === "checkbox" || this.type === "radio" ) { // we don't search against the document in case the element // is disconnected from the DOM - this.buttonElement = this.element.parents().last() - .find( "label[for=" + this.element.attr("id") + "]" ); + var ancestor = this.element.parents().last(), + labelSelector = "label[for=" + this.element.attr("id") + "]"; + this.buttonElement = ancestor.find( labelSelector ); + if ( !this.buttonElement.length ) { + ancestor = ancestor.length ? ancestor.siblings() : this.element.siblings(); + this.buttonElement = ancestor.filter( labelSelector ); + if ( !this.buttonElement.length ) { + this.buttonElement = ancestor.find( labelSelector ); + } + } this.element.addClass( "ui-helper-hidden-accessible" ); var checked = this.element.is( ":checked" ); |