Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Pipeline Summary is blocking the graph #289

Closed
lluunn opened this issue Nov 15, 2018 · 2 comments
Closed

Pipeline Summary is blocking the graph #289

lluunn opened this issue Nov 15, 2018 · 2 comments
Labels
area/frontend

Comments

@lluunn
Copy link

lluunn commented Nov 15, 2018

screen shot 2018-11-15 at 13 52 46

I think summary should not be rendered on top of the graph?
@yebrahim
Copy link
Contributor

We can also add some bottom margin where the hidden summary won't overlap with the graph.
@ajayalfred any preference here?

@ajayalfred
Copy link
Contributor

The hidden summary should be a button. See the two states below:

Summary open
image

Summary closed
image

@rileyjbauer rileyjbauer mentioned this issue Nov 16, 2018
Merged
Linchin pushed a commit to Linchin/pipelines that referenced this issue Apr 11, 2023
@jlewi @k8s-ci-robot
Test utilities for CRDs. (kubeflow#289)
6517f1c 
* Test utilities for CRDs.

* Add a generic wait for condition method for CRDs.

* Fix a bug in setup_ks_app to allow for empty params.

* Fix lint.
magdalenakuhn17 pushed a commit to magdalenakuhn17/pipelines that referenced this issue Oct 22, 2023
@jinchihe @k8s-ci-robot
update kfservice status URL (kubeflow#289)
ed5f743
HumairAK pushed a commit to red-hat-data-services/data-science-pipelines that referenced this issue Mar 11, 2024
@ckadner
Move pytest dependency into Makefile (kubeflow#289)
a83821b 
Resolves kubeflow#288
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
area/frontend
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@yebrahim @ajayalfred @lluunn