-
-
Notifications
You must be signed in to change notification settings - Fork 91
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
Hide Unformatted TypeScript errors / move to after the pretty errors #3
Comments
this is intended. Line 38 in d055b01
|
Hmm. Well, if it's possible, having an option to disable them is much preferred than having a bunch of similar errors in a scrollable small container. For the copy-pasting issue, I believe the error translation feature and also the built-in |
I agree! Would be great to have an option to disable them. Personally I never copy these errors anyway. |
at most, i copy the error number, in this case |
@robertmoura Thank you for opening this issue 💁🏼♀️ So, I'll explore adding a setting that will hide TypeScript's original errors very soon. |
Are extensions able to silence the errors of another extension? |
@Samathingamajig Actually I didn't find any API of VSCode that allows that. Maybe we should address it with a language service plugin like this guy did here? |
Personally, I don't think it's worth it to try to silence the original error. Since formatting restricts the ability to copy/paste the error with the customize one, you get the benefit of being able to easily copy the raw error without going to the error panel. I would imagine silencing the error with a language service plugin would also remove the error from the error panel, as well as hiding it from your extension (unless your extension relies on a separate language server, which I doubt). |
At least it would be cool to have the pretty error first in the tooltip 🤞 |
It was the first on the tooltip a while ago, I'm not sure why it changed. We'll update about it soon 🙏🏼 |
Just to clarify...don't expect it, the language server can cut the original error message, but this will also cause extensions like ErrorLens that rely on diagnostic messages to not work, so I don't think that's the way forward. Still need see VSCode how to improve. In addition, the language server can only control the order of hover messages, but cannot make the hover messages displayed on top of the diagnosis. Sorry for the false hopes. |
Can someone just link to a VSCode repo Issue where we can 👍 some "Let us tweak this" issue? |
A good solution for that would be to hide the original error, but add a button for copying it all to the clipboard. |
read the last sentence |
@RoyiNamir It could be made more prominent, however, it is mentioned on the main screen of the extension. Line 38 in d055b01
(emphasis mine) |
@RoyiNamir A bit rude but I was expecting also only the beautified error. That's why I found this issue. |
This is the issue that will eventually allow us to modify the order of the errors + mute the original error: Every vote or comment could help!@robertmoura Can you add it to the content of the issue please? 🙏🏼 |
This would be great, voting 👍 |
I can confirm that the original "ugly" typescript errors in fact do appear over the "pretty" ones for me as well, feels like they were on top before, so might be as of lately that pretty ones appear below, sadly. For long error messages, this quickly makes the extension near useless, as scrolling the tiny little problems window is far from easy. I would agree that the README should be updated to more accurately show how the errors will be displayed. Hope that VSCode will allow for a way to work around this. It's sad, since the extension is def one of my favourites 🙏🏽 |
Until things will be changed, you can use this hack here: |
Hmm, I am able to copy from the prettified text. macOS. |
I'm curious how. Are you able to copy just texts or also colored code blocks? |
Ahh, now I see the issue! It does seem that at least one of the supporting issues has been upvoted enough, but even though that occurred does not seem like it is getting done, which is shame. |
Really loving the format of these errors! It's a lot easier to read.
The only thing I noticed is that the unformatted errors are still showing up. Was this intended or is there a way to remove them?
Update / TLDR
This is currently intentional but there are plans to improve the experience. This could potentially be improved by:
The issue is that the improvements rely on changes to VS Code. Go upvote this issue if you want these changes!
@yoavbls wrote:
The text was updated successfully, but these errors were encountered: