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

Hilighting broken on themes which use italics #5216

Closed
glfmn opened this issue Dec 19, 2022 · 2 comments
Closed

Hilighting broken on themes which use italics #5216

glfmn opened this issue Dec 19, 2022 · 2 comments
Labels
C-bug Category: This is a bug

Comments

@glfmn
Copy link

glfmn commented Dec 19, 2022

Summary

When setting the styling of comments for a theme, the following results in unexpected styling:

"comment" = { fg = "gray1", modifiers = ["italic"]  }

image

However, it does work when using the following forms:

"comment" = "gray1"
"comment" = { fg = "bg0", bg = "gray1" }
"comment" = { fg = "gray1", modifiers = ["bold"] }

image

Reproduction Steps

Select the gruvbox theme (or any theme which uses italics for comment syntax hilighting).

Create a new theme which inherits from it and override the comment style. Toggle between the themes and observe that themes with modifiers do not have the correct output.

# Modify the gruvbox theme
inherits = "gruvbox"

# Change the look of comments
"comment" = "gray1"

I have not tested anything other than comments, but it could just be an issue with italics in general.

Helix log

~/.cache/helix/helix.log
2022-12-19T13:18:16.648 helix_view::theme [WARN] Theme: malformed hexcode: test

Platform

Linux

Terminal Emulator

alacritty 0.9.0

Helix Version

helix 22.12 (ec9aa66)

@glfmn glfmn added the C-bug Category: This is a bug label Dec 19, 2022
@kirawi
Copy link
Member

kirawi commented Dec 19, 2022

Is this a dupe of #3099?

@glfmn
Copy link
Author

glfmn commented Dec 19, 2022

Yes, didn't think to check if tmux/byobu was the issue!

@glfmn glfmn closed this as completed Dec 19, 2022
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