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

Curly Underlines Don't Work in zellij #6510

Closed
aidandenlinger opened this issue Mar 31, 2023 · 3 comments
Closed

Curly Underlines Don't Work in zellij #6510

aidandenlinger opened this issue Mar 31, 2023 · 3 comments
Labels
C-bug Category: This is a bug

Comments

@aidandenlinger
Copy link

aidandenlinger commented Mar 31, 2023

Summary

When updating to helix 23.03, all the themes were updated to use curly underlines for diagnostic warnings in #5419. These curly underlines are not working when running helix inside of zellij, a terminal multiplexer.

My config.toml is blank.

For the time being this is resolvable by copying the theme file, changing the style of "diagnostic.*" to "line", and using that theme instead when using helix in zellij. Ideally the curly underlines would be displayed, or at least they would not be by default in terminals where they are broken. Not sure if this is a helix or zellij issue.

Running in alacritty by itself:
image

Inside of a zellij session in alacritty:
image

Reproduction Steps

I tried this:

  1. Opened alacritty and opened a zellij session
  • Note: zellij can be quickly downloaded and run with the terminal command at https://zellij.dev/
  1. Opened a project with a diagnostic warning (easy way: create a new rust project with cargo init, remove the ! from println to get a warning)

I expected this to happen:

  • Curly underlines to appear under the warning

Instead, this happened:

  • No underline. The warning is still there and the message appears in the top right when I'm viewing the warning, but no underlines.

Helix log

~/.cache/helix/helix.log

(There's nothing of substance in the log, it's just some vcs warnings because this is a est repo)

2023-03-31T11:30:30.277 helix_vcs [ERROR] PeelToCommit(
    Unborn {
        name: FullName(
            "refs/heads/main",
        ),
    },
)
2023-03-31T11:30:30.277 helix_vcs [ERROR] failed to open diff base for /home/user/Documents/mytest/src/main.rs
2023-03-31T11:30:30.277 helix_vcs [ERROR] PeelToCommit(
    Unborn {
        name: FullName(
            "refs/heads/main",
        ),
    },
)
2023-03-31T11:30:30.277 helix_vcs [ERROR] failed to obtain current head name for /home/user/Documents/mytest/src/main.rs

Platform

Linux

Terminal Emulator

zellij 0.35.2, running in alacritty 0.12.0

Helix Version

helix 23.03 (3cf0372)

@aidandenlinger aidandenlinger added the C-bug Category: This is a bug label Mar 31, 2023
@pascalkuthe
Copy link
Member

zellij likely changes $TERM and therefore makes it impossible to automatically detect the correct terminfo support. You can instead manually enable undercurl support by setting undercurl = true in your config (or temporarily with :set undercurl true)

@aidandenlinger
Copy link
Author

aidandenlinger commented Mar 31, 2023

Thank you for the quick response! I wrote this issue before looking closely at zellij's issues - this seems to be an issue on their side (zellij-org/zellij#1594, PR in progress at zellij-org/zellij#2018). I'll close this since this isn't anything wrong with helix.

@danillos
Copy link
Contributor

danillos commented Nov 8, 2023

There a PR to fix it in Zellij zellij-org/zellij#2918

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

3 participants