-
Notifications
You must be signed in to change notification settings - Fork 105
Should jump to first error even when there are warnings #199
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
Comments
You're right. I'll look into it, thanks! |
Thanks. |
That is intended, at least for now. The reason is that it now uses the JSON output from the compiler to extract the information so that it can show inline phantoms. The compiler cannot currently output both JSON and text. There is some discussion going on about refactoring and extracting the code that is responsible for rendering the text, but I think it will be a long ways off before that is ready. |
The problem is, often the inline error doesn't fit into the width of my sublime window (especially with deeply indented code) when I have two sublime windows side by side which is most of the time, then the inline error is cut off/unreadable and I have to rely on the error output window. |
I've been thinking about that, too. What do you think if the inline message was displayed all the way to the left, instead of aligned with the column of where the error appears? Like this: I think this would make it more readable in the common case, but has the drawback that if the error region is far to the right, then F4 will horizontally scroll past the message. Another thing I've been considering is word-wrapping long messages. There are some cases where rust can spit out some very long messages. However, this has the drawback that the message may include a suggested code replacement, and it may display invalid code if word-wrapped incorrectly. |
I think displaying the error message all the way to the left is a better solution than word wrapping. |
Ah, I always have wordwrapping turned on for code. No horizontal scrolling. |
Can't the inline error be always aligned to the left of the viewport (regardless of horizontal scroll pos)? |
Not that I know of. Sublime only allows you to place it directly below a specific column, or inline. |
Uh oh!
There was an error while loading. Please reload this page.
When I get some warnings and some errors during a build and I press F4, it should jump to the first error (and then step from there).
Currently it jumps to the first warning which is often something trivial like
unused import
.This behavior is annoying, I want to fix the errors first because they have higher priority...
(IIRC it used to be the case that F4 jumps to the first error in the past, but maybe I remember wrong.)
The text was updated successfully, but these errors were encountered: