Skip to content

Commit

Permalink
Controlgroup: Fix refresh method tests by adding text content to button
Browse files Browse the repository at this point in the history
In IE8, the empty elements cause issues. Adding text content makes
sense, since empty elements are useless.
  • Loading branch information
jzaefferer authored and arschmitz committed Oct 8, 2015
1 parent 81c4fdf commit 9ff3d8f
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions tests/unit/controlgroup/methods.js
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@ test( "enable", function( assert ) {
var tests = {
"checkboxradio": "<input type='checkbox'>",
"selectmenu": "<select><option>foo</option></select>",
"button": "<button>"
"button": "<button>button text</button>"
},
orientations = {
"horizontal": [
Expand Down Expand Up @@ -108,7 +108,7 @@ $.each( tests, function( widget, html ) {
// Add a label for each element and then append the element to the control group
for ( i = 0; i < 4; i++ ) {
control = $( html ).attr( "id", "id" + i )
.add( $( "<label>" ).clone().attr( "for", "id" + i ) );
.add( $( "<label>label text</label>" ).clone().attr( "for", "id" + i ) );

controls.push( control );
element.append( control );
Expand Down

0 comments on commit 9ff3d8f

Please sign in to comment.