Arvados

Peter Amstutz wrote:

1. The rounded, padded box around the instructions ("Provide a value...") looks a lot like the rounded, padded input fields, despite being a different color. Perhaps less styling would be better here?

Lose "alert" style, gain italics. Better, I think?

1. "Input for XXX component" seems wordy. How about just the component name with a "label" style?

The word "input" is the name of the script parameter. In the case of "input" it sounds redundant. Now, if a "title" is provided in the pipeline description, use that instead of generic "{paramname} for {componentname} component".

1. With the input field full-width, the text should be right-justified

Which text? I'm not picturing this...

1. The instructions should go above the field, not below

I'm deferring to Bootstrap here, which suggests "bold short label above, longer description/help below". Perhaps giving the pipeline template control over both labels via "title" and "description" helps here too?

1. If we're going to style input field with a big, rounded, padded box, can we use the inline x-editable style instead of the popup?

Sort of. I tried this, but even with a select box we get the "size changes when you click" bug we've run into before with inline mode. Also, you have to click twice: once to go from "static form input" to "x-editable select box" mode, then again to make the thing actually drop down. Rather than get bogged down in this, I'd like to leave it here and make our next move "pop up a full-featured select dialog that lets me search the entire system".

1. The note about the components tab at the bottom should be active, e.g. "Click on the components tab to..." instead of passive.

Fixed.

Also made the form 60% instead of 100% content area width. (Better?)

→ d61f277 (and screenshot attached)

Peter Amstutz wrote:

1. Having an custom title is good. For the default title, I we should just be explicit, e.g. "#{cname} / #{component[:script]} / #{pname.to_s.humanize}" makes totally clear the component, script and parameter that you're setting

See attached. (I also added the words component, script, parameter because otherwise you get enigmas like "bwa-aln / bwa-aln / input".)

→ 140944b