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

Getting underline attribute from syntax hl group returns empty string instead of '1' #17624

Closed
kristijanhusak opened this issue Mar 6, 2022 · 3 comments · Fixed by #17630
Closed
Labels
bug-regression wrong behavior that was introduced in a previous commit (please bisect) highlight

Comments

@kristijanhusak
Copy link
Contributor

Neovim version (nvim -v)

NVIM v0.7.0-dev+1194-g24d56116f

Vim (not Nvim) behaves the same?

no, vim 8.2.4464

Operating system/version

Linux

Terminal name/version

Kitty

$TERM environment variable

xterm-kitty

Installation

Built from source

How to reproduce the issue

nvim --clean
:hi UnderlineTest gui=underline
:echo synIDattr(synIDtrans(hlID('UnderlineTest')), 'underline')

Expected behavior

Expected to get 1 returned.

This also properly works in stable versions of Neovim since 0.5.0.

Figured this issue from the tests in nvim-orgmode: https://github.com/nvim-orgmode/orgmode/runs/5439299419?check_suite_focus=true#step:7:96

Link to line where the test fails: https://github.com/nvim-orgmode/orgmode/blob/master/tests/plenary/colors/colors_spec.lua#L79

Actual behavior

Getting the empty string.

@kristijanhusak kristijanhusak added the bug issues reporting wrong behavior label Mar 6, 2022
@clason clason added highlight bug-regression wrong behavior that was introduced in a previous commit (please bisect) and removed bug issues reporting wrong behavior labels Mar 6, 2022
@jamessan
Copy link
Member

jamessan commented Mar 6, 2022

Fallout from @kchibisov's PR. We should be testing the properties individually, so this type of thing gets caught.

@kchibisov
Copy link
Contributor

@jamessan I'd assume the tests should go into functional/terminal/highlight_spec.lua?

@jamessan
Copy link
Member

jamessan commented Mar 6, 2022

That or functional/ex_cmds/highlight_spec.lua would work.

kchibisov added a commit to kchibisov/neovim that referenced this issue Mar 6, 2022
This commit fixes regression introduced in c365de1 when checking for
highlight attribute for underline was returning '0' when it was present

Fixes neovim#17624.
kchibisov added a commit to kchibisov/neovim that referenced this issue Mar 6, 2022
This commit fixes regression introduced in c365de1 when checking for
highlight attribute for underline was returning '0' when it was present

Fixes neovim#17624.
dmitmel pushed a commit to dmitmel/neovim that referenced this issue Apr 16, 2022
This commit fixes regression introduced in c365de1 when checking for
highlight attribute for underline was returning '0' when it was present

Fixes neovim#17624.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug-regression wrong behavior that was introduced in a previous commit (please bisect) highlight
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants