-
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
[UI] Check all UI elements that could be hidden due to long content #2897
Comments
#3677 reported one place: parameter values in input/outputs tab |
Found another issue, in pipeline version selector, version could be too long. |
@Bobgy: Reopened this issue. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
Thanks for the fix #3747. We've realized recently that it's necessary for user to be able to copy URL of artifact on Artifacts page or in "Output artifacts" section of "Input/Output" tab on Run Details page (since artifact URL itself is not visible and link that can be copied is to unarchive/download endpoint). The button is supposed to be like: So, what do you think about button for copying to clipboard? |
@anatolyzolotar Thanks for the suggestion! UI now shows all urls in their complete form. Does it work for your use-case? or do you suggest a copy button is easier to use? Sounds good to me if we add a copy button. I don't currently have bandwidth to implement that because 1.0 release is coming. Can you create an issue capturing the idea? and PRs welcomed |
@Bobgy , yes, complete form of urls will allow user to copy artifact's url (not a link to unarchive/download endpoint) and use it somewhere else for access. (Off topic: can you tell me where I can find any roadmap for 2020 or any kind of release schedule? I couldn't find it anywhere. This fix is crucial for us on this week and, if as far as I understand, we have to the clone repo and build our own docker image with this fix added locally). Regarding proposal: for me it seems like shortened urls (with copy to clipboard) are better option, at least because we also may need to get artifacts url (instead of download link) on Artifacts List page. Urls there are rendered in cells and showing complete form will spoil grid layout. On the other hand I'm not sure if it's doable there since urls are rendered differently (with different components, at the first glance). Will look deeper later. I will create issue and will try to add PR for it. |
@anatolyzolotar
We do bi-weekly release, and you can get images built for any commit in master in gcr.io/ml-pipeline-test/frontend:COMMIT_SHA. |
Thanks for clarifying that. Sounds like a better UX. Of course these are achievable (KFP's UI is implemented with React, so we have all the flexibility here), we'd welcome contributions. |
This is a long standing general issue.
Many UI elements may be partially hidden due to long content. Let's examine our UI and give a workaround for all cases that really needs a way to read full content.
Usually, we should just add a tooltip on hover for full text. e.g. like #2894
The text was updated successfully, but these errors were encountered: