Skip to content
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

Inline diagnostics are invisible when very long and should be partly shown #12121

Open
louis-thevenet opened this issue Nov 24, 2024 · 3 comments
Labels
A-helix-term Area: Helix term improvements C-bug Category: This is a bug

Comments

@louis-thevenet
Copy link

Summary

When inline diagnostics are longer than the number of lines showed by Helix, and I edit a line below the error line. Inline diagnostics disappear instead of being partly showed, leaving a long blank space.

Reproduction Steps

asciicast

You can reproduce it on line 39 of the config.rs file in this branch : https://github.com/louis-thevenet/vault-tasks/tree/helix-broken-inline-diagnostics

Helix log

~/.cache/helix/helix.log
please provide a copy of `~/.cache/helix/helix.log` here if possible, you may need to redact some of the lines

Platform

Linux

Terminal Emulator

Kitty

Installation Method

flake

Helix Version

Helix 24.7 (dc941d6)

@louis-thevenet louis-thevenet added the C-bug Category: This is a bug label Nov 24, 2024
@kirawi kirawi added the A-helix-term Area: Helix term improvements label Nov 26, 2024
@nik-rev
Copy link
Contributor

nik-rev commented Dec 23, 2024

Use zz, zt or zb to be able to see them fully

@louis-thevenet
Copy link
Author

From what I understand, when the cursor is past the line generating the large diagnostics, the virtual lines of the diagnostics are rendered and this pushes the error line up.

If the error line is pushed too high (outside of the viewport), the insert_virtual_lines still returns space for the diagnostics, but the render_virtual_lines function doesn't render those and that's why we get the blank space

@justinlovinger
Copy link

justinlovinger commented Jan 6, 2025

Use zz, zt or zb to be able to see them fully

That is a video in the opening post, not an image. The first frame does not show the issue. Here it is in images:

2025_01_06t12_10_12
2025_01_06t12_10_15
2025_01_06t12_10_17

Note, the second line also has a long error, which is not displayed at all. The editor also lags significantly past that last error.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-helix-term Area: Helix term improvements C-bug Category: This is a bug
Projects
None yet
Development

No branches or pull requests

4 participants