Skip to content

Add a link to create pipeline from dashboard when grouped by pipeline groups. (#5715)#5885

Merged
GaneshSPatil merged 1 commit intogocd:masterfrom
varshavaradarajan:create-pipeline-from-dashboard
Mar 25, 2019
Merged

Add a link to create pipeline from dashboard when grouped by pipeline groups. (#5715)#5885
GaneshSPatil merged 1 commit intogocd:masterfrom
varshavaradarajan:create-pipeline-from-dashboard

Conversation

@varshavaradarajan
Copy link
Contributor

#5715

screen shot 2019-02-19 at 11 57 44 pm

Um, I don't know if there is a better way to show the add icon.
cc @naveenbhaskar

@GaneshSPatil
Copy link
Contributor

@varshavaradarajan --
Wondering if the '+' is the best icon to indicate create pipeline within the group operation.

Few questions:

  1. + icon shouldn't be confusing. What does it mean? Create a pipeline within the group? Or Create a new pipeline group itself?
  2. Allowing to create new pipeline within a group, but not for environments. Will it be consistent?

\cc: @naveenbhaskar

@varshavaradarajan
Copy link
Contributor Author

+ icon shouldn't be confusing. What does it mean? Create a pipeline within the group? Or Create a new pipeline group itself?

If its specified along with the group name, it doesn't mean creating a new pipeline group, right?

Allowing to create new pipeline within a group, but not for environments. Will it be consistent?

Why are we aiming for consistency in this matter? Pipeline group and environment are 2 different entities. A pipeline must always belong to a pipeline group while a pipeline can be optionally part of some environment. Why is it not okay to display options that make sense for a pipeline group if it means improving the UX? If you think that doing this is not necessary, feel free to close the PR, though.

@maheshp
Copy link
Contributor

maheshp commented Feb 21, 2019

If you think that doing this is not necessary, feel free to close the PR, though.

@varshavaradarajan IMO, having an ability to create a pipeline on dashboard would be useful over going to a different page to do the same. We just need to decide on the right UX for this.

@arvindsv
Copy link
Member

I always came up with this in my head, when I thought of the ability to add a pipeline to a group:

2019_02_21_12-18-23

@ketan
Copy link
Member

ketan commented Feb 21, 2019 via email

@varshavaradarajan
Copy link
Contributor Author

varshavaradarajan commented Feb 25, 2019

@arvindsv - should that add box be shown only for admins or be shown to everyone with the add button disabled for view users?

Also, are there plans to make a dashboard group expand/collapse? Because, having the add box in that case may not be of much use as it is still the same number of interactions as clicking on the group name from the dashboard and then clicking on "create a pipeline within this group" in /go/admin/pipelines?

@varshavaradarajan varshavaradarajan force-pushed the create-pipeline-from-dashboard branch from e19339a to d547813 Compare February 25, 2019 06:51
@arvindsv
Copy link
Member

@arvindsv - should that add box be shown only for admins or be shown to everyone with the add button disabled for view users?

I think it should show only for admins. /cc @naveenbhaskar Thoughts?

Also, are there plans to make a dashboard group expand/collapse? Because, having the add box in that case may not be of much use as it is still the same number of interactions as clicking on the group name from the dashboard and then clicking on "create a pipeline within this group" in /go/admin/pipelines?

True, but it's still an extra page load in that case. I think this box is still useful.

@varshavaradarajan varshavaradarajan force-pushed the create-pipeline-from-dashboard branch from d547813 to 3f2ea11 Compare February 26, 2019 20:40
@naveenbhaskar
Copy link
Contributor

if we add a blank pipeline box at the end of pipeline group, the problems i see is

  1. If there are a lot of pipelines in a group, user has to scroll a lot to reach the end and click on it.
  2. There was a proposal for expand-collapse the pipeline groups, so this is not easily reachable.

The best place for adding a pipeline to the group is pipeline group header itself. it is easily accessible and noticeable and it should only show for people who can add pipelines.

image

This design will be consistent with design for admin-> pipelines

image

@ibnc
Copy link
Contributor

ibnc commented Feb 27, 2019

Would the possibility of combining @arvindsv @naveenbhaskar's designs be out of the question?

@arvindsv`s design would be really helpful/intuitive for new users, and @naveenbhaskar's design would add benefit for instances with a great number of pipelines.

thoughts?

@ketan
Copy link
Member

ketan commented Feb 28, 2019

+1 to what @ibnc said.

@arvindsv
Copy link
Member

I like that suggestion, if it makes sense from a design perspective.

@varshavaradarajan varshavaradarajan force-pushed the create-pipeline-from-dashboard branch from 3f2ea11 to c859220 Compare March 2, 2019 03:27
@varshavaradarajan
Copy link
Contributor Author

@naveenbhaskar - can you please review this PR when you're free? Thank you!

screen shot 2019-03-01 at 6 58 22 pm

@varshavaradarajan
Copy link
Contributor Author

@arvindsv, @ibnc - by the way, just wanted to point out that a pipeline group which has no pipelines is not shown on the dashboard. So, this is only for making it easier to add pipelines to groups which already has at least one pipeline. Not sure if you want to change that as part of improving the ux for new users.

@varshavaradarajan varshavaradarajan force-pushed the create-pipeline-from-dashboard branch from c859220 to 2dd9dd9 Compare March 2, 2019 06:21
@streisguth
Copy link

Hi all, I know I'm a little late to this issue but I do not think that having both buttons is a good design solution. I am anticipating that having the same action in two similar places will be confusing to a user, as I would expect each of those buttons to have an action different from one another.

My preference would be to have the button in the top of the pipeline group only, as it is the design that covers the use cases of having either a small number of pipelines OR a large number of pipelines.

@GaneshSPatil
Copy link
Contributor

+1 to @streisguth
Keeping 2 buttons to perform the same operation is confusing.

Can we do smart positioning?
If there are more than X pipelines (lets say 15), use the button at the top of the pipeline group.
And if there are less than X pipelines, use the '+' pipeline box?

Note:
Also, with this approach, the sudden change in the feature usability might cause confusion.

@varshavaradarajan varshavaradarajan force-pushed the create-pipeline-from-dashboard branch from 2dd9dd9 to 524e70a Compare March 12, 2019 20:17
@naveenbhaskar
Copy link
Contributor

+1 to @streisguth .. There should not be two action element for the same action.

@varshavaradarajan
Copy link
Contributor Author

@naveenbhaskar - I've already made changes suggested by @streisguth . The New Pipeline button shows up in the dashboard group header like your screenshot. Do you mind reviewing the changes so we can get this merged?

@naveenbhaskar
Copy link
Contributor

sure @varshavaradarajan

@naveenbhaskar
Copy link
Contributor

@varshavaradarajan .. I have verified it and it is good to go from UI point of view.

@GaneshSPatil
Copy link
Contributor

The final UI screenshot:
Screen Shot 2019-03-25 at 12 40 35 PM

\cc: @naveenbhaskar @varshavaradarajan @rajiesh @adityasood

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants