8000 Text in the status line is squashed onto one line · Issue #2301 · dense-analysis/ale · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Text in the status line is squashed onto one line #2301

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

Closed
keysmashes opened this issue Feb 16, 2019 · 5 comments
Closed

Text in the status line is squashed onto one line #2301

keysmashes opened this issue Feb 16, 2019 · 5 comments
Labels

Comments

@keysmashes
Copy link

When I'm writing Rust, the errors in the status line get concatenated into one line:

image

but it's kind of confusing to have to work out where the first line of the error message ends and the other lines begin every time I want to fix an error. Compare with :ALEDetail:

image

Would it be possible to separate the lines of the error message somehow in the status line? Even just putting a space between them would be an improvement, or it would be neat if it were possible to apply different styles to different parts of the error message (e.g. display the first line in bold, or everything except the first line in a dimmer colour).

@RyanSquared
Copy link
Member

I've seen this happen enough times with various Rust things that I think we should put up a handler for all Rust messages to be wrapped in this - i.e. if there's three lines, just put in a : like what one of the tools does internally. This makes it more consistent from a UI perspective as all linters would give the same output.

@w0rp
Copy link
Member
w0rp commented Feb 17, 2019

Putting a space between them makes sense. They have to be put into one line and truncated so they don't create "Press Enter to continue" prompts as you move your cursor around, but we can add a space.

@shuwens
Copy link
Contributor
shuwens commented Apr 5, 2019

Putting a space between them makes sense. They have to be put into one line and truncated so they don't create "Press Enter to continue" prompts as you move your cursor around, but we can add a space.

This sounds great. Would it be trivial to implement? If so I can try to tackle it and push a PR.

I also want to point out that I think we need ALEDetail/virtualtext/status line msg to have a way to format them automatically anyway.

@w0rp
Copy link
Member
w0rp commented Apr 8, 2019

This shouldn't be hard to fix, split the lines and join them together again. Save different text in the text and detail keys.

@w0rp
Copy link
Member
w0rp commented Apr 15, 2019

Now newlines are replaced with spaces in the messages we echo.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants
0