-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Comments
On the screen shot above, I press the back button and nothing happens. This run was created from a notebook. |
Any console errors? |
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 |
Yes, this is why we have this comment here: pipelines/frontend/src/components/Toolbar.tsx Line 139 in 4550513
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. |
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? |
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. |
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. |
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. |
Oh wow, yeah that's a good catch. So this hack would work from the experiment details page but not the run details page. |
Yup, that's the reason GCP disables it. |
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. |
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. |
* Add support for ExitHandlers in UI * Add default empty arrays * Change label and background color for Static Graph
The text was updated successfully, but these errors were encountered: