incubator-devlake
incubator-devlake copied to clipboard
apache
[Feature][config-ui] Improve usability of the "Task failed: ..." popup
Search before asking
- [X] I had searched in the issues and found no similar feature requirement.
Use case
- I want the "Task failed..." popups to have good readability, even when the information is a verbose system error log.
- I want to be able to copy paste the information in the "Task failed..." popups
- When the displayed information is a log from a system error I want to know when it happened
Description
The screenshot under documents where the popup panel I mention is: it appears when you hover over "Task failed: hover to view the reason"
Today when I hover over and the reason is an error in the execution I am met with this kind of stuff:
So back to my 3 use case requests:
- Readability when system errors occur is difficult. I think it's satisfying that you reproduce the relevant log statement, as it allows me to dig in the logs server side to find more information. But I suggest that you a) limit the maximum size of the popup and add some kind of +/- button to maximize and minimize it" and more importantly b) format the log so newlines are correctly placed. The second change in particular should be trivial and would really improve readability.
- "copy pasting" is impossible today because the popup disappear when you move the mouse. You can ofc use the developer tools to get the message but it unnecessarily complicated. I propose you either a) make the link clikable, and when it's clicked it would stay unhidden, giving the user time to copy/paste or b) add a "copy" icon close to the text which when clicked would copy the text to the clipboard.
- finally you must add the time at which the log occured, that's a crucial piece of information.
Consider my proposals as illustrative - there are many ways to solve this but it's usually simpler to be specific and let people adjust than not saying anything.
Related issues
No response
Are you willing to submit a PR?
- [ ] Yes I am willing to submit a PR!
Code of Conduct
- [X] I agree to follow this project's Code of Conduct
Hi @renflo, thank you for your report. This is just a simple error message, if you want to get the complete log information, you can download the error log. And @Startrekzky do you have any advice?
I'm aware it's a simple error message and it's fine. It's just that when the message displayed is huge it's not very usable. A few adjustments like the ones I mention would help a lot. I'm ofc not proposing to replace looking up the logs from the server :)
@klesh
Maybe we should limit the length of charaters of API response's err message
Hi @renflo You don't need to get logs from the server; you can download logs from the config-ui.
Ok, thanks, did not see that option, it's quite "far" away from the link which creates the popup.
Perhaps the button could be added close to the link i mentioned? There is something not ideal about usability here.
This issue has been automatically marked as stale because it has been inactive for 60 days. It will be closed in next 7 days if no further activity occurs.
![github-actions[bot] avatar](https://avatars.githubusercontent.com/in/15368?v=4 "Published by a pub.dev verified publisher"){kind=small}
This issue has been automatically marked as stale because it has been inactive for 60 days. It will be closed in next 7 days if no further activity occurs.
This issue has been automatically marked as stale because it has been inactive for 60 days. It will be closed in next 7 days if no further activity occurs.
This issue has been automatically marked as stale because it has been inactive for 60 days. It will be closed in next 7 days if no further activity occurs.
This issue has been automatically marked as stale because it has been inactive for 60 days. It will be closed in next 7 days if no further activity occurs.