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

fix(highlight.vim): use default value for compose_hlgroup #3514

Merged
merged 1 commit into from
Dec 20, 2021

Conversation

kevinhwang91
Copy link
Contributor

If two hightlight groups both are empty or single reverse attribute,
coc#highlight#compose_hlgroup can't build new highlight group.

Such as:

hi Group1 gui=reverse
hi clear Group2
echo coc#highlight#compose_hlgroup('Group1', 'Group2')

will throw E411: highlight group not found: FgGroup1BgGroup2

If two hightlight groups both are empty or single reverse attribute,
`coc#highlight#compose_hlgroup` can't build new highlight group.

Such as:
```vim
hi Group1 gui=reverse
hi clear Group2
echo coc#highlight#compose_hlgroup('Group1', 'Group2')
```
will throw E411: highlight group not found: FgGroup1BgGroup2
@codecov
Copy link

codecov bot commented Dec 18, 2021

Codecov Report

Merging #3514 (d3c8e1b) into master (a1688fc) will increase coverage by 0.00%.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #3514   +/-   ##
=======================================
  Coverage   74.74%   74.75%           
=======================================
  Files         191      191           
  Lines       19480    19480           
  Branches     4360     4360           
=======================================
+ Hits        14561    14562    +1     
  Misses       3154     3154           
+ Partials     1765     1764    -1     
Impacted Files Coverage Δ
src/completion/index.ts 71.77% <0.00%> (-0.27%) ⬇️
src/list/ui.ts 91.88% <0.00%> (ø)
src/list/basic.ts 88.19% <0.00%> (+0.69%) ⬆️
src/model/task.ts 92.10% <0.00%> (+2.63%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a1688fc...d3c8e1b. Read the comment docs.

@chemzqm chemzqm merged commit 8175e7a into neoclide:master Dec 20, 2021
@kevinhwang91 kevinhwang91 deleted the fallback4composehl branch December 20, 2021 19:56
chemzqm added a commit that referenced this pull request Dec 21, 2021
077e470 fix(format): onTypeFormatting position (#3472)
d5a4ee8 fix(float): check nvim 0.5.1 for zindex
f59f77c fix(source-language): fix range of textEdit
b532597 chore(handler): fix types for openCommand
4913238 chore(doc): add coc.preferences.semanticTokensHighlights
cbd326b feat(typings): export SemanticTokensBuilder (#3357)
2ff2029 fix(api): preserve register content
19446d6 feat(sources): avoid \ as trigger character
69e8750 fix: typos in schema & docs (#3446)
8175e7a fix(highlight.vim): use default value for compose_hlgroup (#3514)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants