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.

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

Back-button not working when creating a run from a notebook #221

Closed
vicaire opened this issue Nov 12, 2018 · 12 comments
Closed

Back-button not working when creating a run from a notebook #221

vicaire opened this issue Nov 12, 2018 · 12 comments

Comments

@vicaire
Copy link
Contributor

vicaire commented Nov 12, 2018

screenshot from 2018-11-12 15-44-33

@vicaire
Copy link
Contributor Author

vicaire commented Nov 12, 2018

On the screen shot above, I press the back button and nothing happens.

This run was created from a notebook.

@yebrahim
Copy link
Contributor

Any console errors?

@rileyjbauer
Copy link
Contributor

No console errors. The issue is that we're landing directly on that page, with no history.

In this case, the back arrow should either not exist, or take the user back to the ExperimentDetails page. Both will result in behavior that is not quite the same as what one would encounter if working solely through the UI

@yebrahim
Copy link
Contributor

Yes, this is why we have this comment here:

// Need to handle this for when browsing back doesn't make sense
.

I've noticed GCP console disables the button in this case. I don't like this but I guess it makes most sense if the button is just browser back.

@vicaire
Copy link
Contributor Author

vicaire commented Nov 15, 2018

My expectation for this one would be for the back arrow (which I am used to) to be present and to take me back to the experiment. WDYT?

@yebrahim
Copy link
Contributor

After debating this for a while, we thought the best course of action is browser back, which is least controversial/confusing, and used by GCP console. We just need to workaround this corner case.

@ajayalfred
Copy link
Contributor

Can you default to whatever is the last link in the breadcrumb? Anytime browser history is absent and you have to display a back button, make it link to the last link in the breadcrumb above.

@rileyjbauer
Copy link
Contributor

We could do that, but note that would lead to a weird situation in the case described by you and Pascal where you start on the run page, go back up to the experiment page, and then go back to the run page, and just bounce back and forth between them if you keep clicking the back arrow.

@ajayalfred
Copy link
Contributor

Oh wow, yeah that's a good catch. So this hack would work from the experiment details page but not the run details page.
I guess the best course of action is to display the button but disable it. That way a user can still use breadcrumbs to navigate around, and reenable the button when there's browser history.
Do you have any other solutions?

@yebrahim
Copy link
Contributor

Yup, that's the reason GCP disables it.
The other solution is to take it out completely.

@vicaire
Copy link
Contributor Author

vicaire commented Nov 16, 2018

As a user, my expectation is that the behavior of the back arrow (if wee keep it) is consistent and not the same as the browser back button.

So if takes me once from the run page to the experiment page, I would expect it to always do this, no matter the history.

@ajayalfred
Copy link
Contributor

In theory it sounds right that the back button take you up one level instead of browser back – however in our daily testing and usage, browser back is actually the most useful resolution for the need to go back. Because more than likely, a user is not always traversing the hierarchy to complete a task. When we add filtering, browser back becomes important to resubmit the filter string to see the same results again.

For this particular edge case (when no browser history is present), disabling the button sounds good.

HumairAK pushed a commit to red-hat-data-services/data-science-pipelines that referenced this issue Mar 11, 2024
* Add support for ExitHandlers in UI

* Add default empty arrays

* Change label and background color for Static Graph
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants