Commit 58cf2518 authored by Eric Windham's avatar Eric Windham Committed by Kenny Hall

fixes issue#3289 where optionRepeaters were not setting the selected option correctly (#3290)

parent 129696ff
......@@ -160,13 +160,14 @@ define( ['views/app/drawer/optionRepeaterError'], function( ErrorView ) {
if( 'undefined' == typeof that.options.columns[ column ] ) return;
var select = document.createElement( 'select' );
var html = '';
_.each( that.options.columns[ column ].options, function( option ){
var optionNode = document.createElement( 'option' );
optionNode.selected = ( value == option.value );
optionNode.value = option.value;
optionNode.label = option.label;
if ( value === option.value ) {
optionNode.setAttribute( 'selected', 'selected' );
}
optionNode.setAttribute( 'value', option.value );
optionNode.setAttribute( 'label', option.label );
select.appendChild( optionNode );
});
......
This diff is collapsed.
This source diff could not be displayed because it is too large. You can view the blob instead.
This diff is collapsed.
This diff is collapsed.
This source diff could not be displayed because it is too large. You can view the blob instead.
File mode changed from 100755 to 100644
File mode changed from 100644 to 100755
This source diff could not be displayed because it is too large. You can view the blob instead.
This diff is collapsed.
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment