Story #3088

Workbench UI improvements: Run this pipeline on template should bring up project chooser.

Added by Radhika Chippada about 5 years ago. Updated about 5 years ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Radhika Chippada
Category:
-
Target version:
Start date:
06/26/2014
Due date:
% Done:

100%

Estimated time:
(Total: 0.00 h)
Story points:
1.0

Subtasks

Task #3102: Review 3088-project-chooser-on-run-pipeline-templateResolvedRadhika Chippada

Associated revisions

Revision f0ab1bd4
Added by Radhika Chippada about 5 years ago

closes #3088
Merge branch '3088-project-chooser-on-run-pipeline-template'

History

#1 Updated by Radhika Chippada about 5 years ago

  • Assigned To set to Radhika Chippada

#2 Updated by Radhika Chippada about 5 years ago

  • Status changed from New to In Progress

#3 Updated by Radhika Chippada about 5 years ago

refs #2836

#4 Updated by Radhika Chippada about 5 years ago

- Updated the "Run this pipeline" button in the template display page

- Updated the "Run" button in the pipeline_templates in the recent templates view page

- Updated the integration test to work with the new UI

- Also, included in this story are the updates for "Show date-or-time instead of age" from #2836

#5 Updated by Brett Smith about 5 years ago

This branch looks good. Thanks for an easy review. I just have one small comment and one suggestion, both about icons:

  • The new button on the pipeline_templates#show page uses the fa-folder icon. While I understand the logic behind this, the relationship with the text "Run this pipeline" seems unclear. Perhaps fa-play would be more suitable? That's what we use on pipeline_templates#index.
  • This second issue originated before your branch, but while we're changing it: it looks like most of our buttons have icons on the left, but the Run button on pipeline_templates#index has fa-play on the right. Do you think it'd look more consistent with the rest of the UI if we moved it to the left too?

The code is clean, so I'm happy for this to get merged, with or without patches if you feel strongly about the current icon situation. Thanks.

#6 Updated by Radhika Chippada about 5 years ago

  • Status changed from In Progress to Resolved

Applied in changeset arvados|commit:f0ab1bd48f45470ffb6fefafa713c51347f5a194.

Also available in: Atom PDF