There are three failing API tests that seem like legimate regressions. https://jenkins.galaxyproject.org/job/docker-api/1494/#showFailuresLink

Thanks for looking into it.

@guerler we need a rebase here and have 4 failing tests.

@galaxybot test this
The 4 failing TS tests are unrelated to this PR, they should be fixed with this run.
edit: not fixed by rerun but definitely unrelated

• it would be nice if the entire tool-header would be click able and open/close a tool
• on a large screen the tools are on the right side of the main frame ... center or justify would be nice
• on the other hand, now that we have more space left and right to the tool boxes, some arrows left and right indicating from which input belongs to which output would be great

Usability feels very good! Great work Aysam!

Anyone wants to test it? Use this prepared Docker Image:

docker run -p 8080:80 -i -t bgruening/galaxy-stable:guerler

@guerler This is the fix for the conflict. I tried to create a PR against your branch but because it is way behind dev (and in conflict :D) it caused trouble so I am sending it at least like this.

guerler@3eef069#diff-fae872b63d46cbf39f4d2749c699f8e0R12

The whole thing with dev merged is in https://github.com/martenson/galaxy/commits/fix-js-imports (the fix-js-imports branch)

Also I am getting

File '/Users/marten/devel/git/galaxy/database/compiled_templates/workflow/run.mako.py', line 91 in render_body
  step_model[ 'step_id' ] = step.id
TypeError: 'NoneType' object does not support item assignment

when trying to access the WF run form (no matter if the dev-merged version or @guerler's)
What am I doing wrong?

@guerler and I discussed this and have decided to mirror the effort of the new tool form (keeping it optional by default initially). We will merge this today (for the exposure and because "97 changed files with 1,374 additions and 1,219 deletions." stales quickly) and then create an issue with things that need to be done before this can be the primary workflow form.

@jmchilton 👍