Actions
Idea #3088
closedWorkbench UI improvements: Run this pipeline on template should bring up project chooser.
Status:
Resolved
Priority:
Normal
Assigned To:
Radhika Chippada
Category:
-
Target version:
Start date:
06/26/2014
Due date:
Story points:
1.0
Updated by Radhika Chippada over 10 years ago
- Assigned To set to Radhika Chippada
Updated by Radhika Chippada over 10 years ago
- Status changed from New to In Progress
Updated by Radhika Chippada over 10 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
Updated by Brett Smith over 10 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.
Updated by Radhika Chippada over 10 years ago
- Status changed from In Progress to Resolved
Applied in changeset arvados|commit:f0ab1bd48f45470ffb6fefafa713c51347f5a194.
Actions