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

Diagnostics can overlay warnings addressed by cursor #12197

Closed
Rudxain opened this issue Dec 5, 2024 · 1 comment
Closed

Diagnostics can overlay warnings addressed by cursor #12197

Rudxain opened this issue Dec 5, 2024 · 1 comment
Labels
C-bug Category: This is a bug

Comments

@Rudxain
Copy link
Contributor

Rudxain commented Dec 5, 2024

Summary

(inline?) diagnostics can block the view of arbitrary text, including the source of the warn/error itself. This makes it hard for users to fix it.

Workarounds:

  • Use low zoom level (small font): Not always possible, since the source of the warn/err can be at the top of the file.
  • Move the view-port, and/or add lines before the source of error: requires reverting the change manually after fixing the warn/err

Reproduction Steps

The following example screenshots are contrived, but the general problem happens frequently for me (high zoom):
Cursor not on error: no diagnostic

Cursor on error (underline): diagnostic now makes the entire line invisible

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

GNOME Terminal 3.54.2 using VTE 0.78.2 +BIDI +GNUTLS +ICU +SYSTEMD for GNOME 47

Installation Method

AppImage from releases page

Helix Version

helix 24.7 (079f544)

@Rudxain Rudxain added the C-bug Category: This is a bug label Dec 5, 2024
@the-mikedavis
Copy link
Member

I don't think those are inline diagnostics - #6417 was merged right after the 24.07 release so if you're using the latest stable it shouldn't be possible to use them. (The screenshot looks like non-inline-diagnostics: the text is right-aligned.)

So this may be a duplicate of #1462

@Rudxain Rudxain closed this as not planned Won't fix, can't repro, duplicate, stale Dec 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-bug Category: This is a bug
Projects
None yet
Development

No branches or pull requests

2 participants