diff --git a/rundeckapp/grails-app/i18n/messages.properties b/rundeckapp/grails-app/i18n/messages.properties index 600eb4291f7..f4ebdaf9fe5 100644 --- a/rundeckapp/grails-app/i18n/messages.properties +++ b/rundeckapp/grails-app/i18n/messages.properties @@ -248,7 +248,7 @@ execution.show.mode.Compact.desc=View all output collated by node #form text form.option.multivalued.description=Allow multiple input values to be chosen. -form.option.delimiter.description=Delimiter will be used to join all input values. +form.option.delimiter.description=Delimiter will be used to join all input values. Can be any string: ' ' (space), ',' (comma), etc... # API Messages api.error.api-version.required=API Version not specified diff --git a/rundeckapp/grails-app/views/framework/_optionValuesSelect.gsp b/rundeckapp/grails-app/views/framework/_optionValuesSelect.gsp index 651ca0f2ffb..9ce75ef2f5a 100644 --- a/rundeckapp/grails-app/views/framework/_optionValuesSelect.gsp +++ b/rundeckapp/grails-app/views/framework/_optionValuesSelect.gsp @@ -66,7 +66,7 @@ <%-- variable input text fields --%>
- + New Value…
diff --git a/rundeckapp/web-app/css/main.css b/rundeckapp/web-app/css/main.css index 1504c155a17..1afb9ebd347 100644 --- a/rundeckapp/web-app/css/main.css +++ b/rundeckapp/web-app/css/main.css @@ -312,14 +312,29 @@ h1 { .optname.required{ font-weight:bold; } -.optionvaluemulti{ - margin:4px 0; +.optionvaluemulti label{ + padding: 2px 4px; +} +.optionvaluemulti label{ + display:block; +} +.optionvaluemulti label:hover{ + background: #e3edfa; +} + +.optionvaluemulti span.action { + display:block; + margin:4px; +} +.optionvaluemulti div.varinput { + margin:4px; } .optionmultiarea{ overflow-y: auto; - max-height: 8em; + max-height: 11em; border: 1px solid #ddd; - padding: 4px; + padding: 0; + min-width: 200px; } .enforceSet{ text-align:center; diff --git a/rundeckapp/web-app/js/executionOptions.js b/rundeckapp/web-app/js/executionOptions.js index bb1c78af620..3829cf5c799 100644 --- a/rundeckapp/web-app/js/executionOptions.js +++ b/rundeckapp/web-app/js/executionOptions.js @@ -57,6 +57,9 @@ var ExecutionOptions = { div.addClassName('optionvaluemulti'); div.setStyle({'opacity':'0'}); + var divwrap = new Element('div'); + divwrap.addClassName('varinput'); + var inpu = new Element('input'); inpu.setAttribute("type", "checkbox"); inpu.setAttribute("name", "extra.option." + name); @@ -75,8 +78,9 @@ var ExecutionOptions = { Event.observe(inpu2, 'change', ExecutionOptions.multiVarInputChangeHandler.curry(inpu)); Event.observe(inpu2, 'keydown', ExecutionOptions.multiVarInputKeydownHandler.curry(inpu)); - $(div).appendChild(inpu); - $(div).appendChild(inpu2); + $(divwrap).appendChild(inpu); + $(divwrap).appendChild(inpu2); + $(div).appendChild(divwrap); $(inputarea).insert({top:div}); $$('#' + name + '_state span.reqwarning').each(Element.hide); Try.these(