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(float): missing default highlight for title #28617

Merged
merged 1 commit into from
May 25, 2024

Conversation

glepnir
Copy link
Member

@glepnir glepnir commented May 3, 2024

Problem: there is missing default title highlight when highlight not defined in title text chunk.

Solution: when attr is not set use default title highlight group.

@glepnir glepnir added the float floating windows label May 3, 2024
@glepnir glepnir requested a review from zeertzjq May 3, 2024 07:56
Copy link
Member

@zeertzjq zeertzjq left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is not documented.

@zeertzjq
Copy link
Member

zeertzjq commented May 3, 2024

I'm also not sure if this is desired. This is not backward-compatible with v0.9.

@glepnir
Copy link
Member Author

glepnir commented May 3, 2024

When no highlight is configured in chunk , assume they use the default title highlight like string.

@glepnir glepnir requested a review from bfredl May 3, 2024 08:13
runtime/doc/api.txt Outdated Show resolved Hide resolved
src/nvim/drawscreen.c Outdated Show resolved Hide resolved
src/nvim/decoration.c Outdated Show resolved Hide resolved
@justinmk justinmk added the needs:response waiting for reply from the author label May 11, 2024
@glepnir glepnir force-pushed the border_hl branch 2 times, most recently from c1b6c43 to d26d42c Compare May 12, 2024 05:52
src/nvim/drawscreen.c Outdated Show resolved Hide resolved
src/nvim/drawscreen.c Outdated Show resolved Hide resolved
src/nvim/api/win_config.c Outdated Show resolved Hide resolved
src/nvim/api/win_config.c Outdated Show resolved Hide resolved
runtime/doc/api.txt Outdated Show resolved Hide resolved
runtime/doc/api.txt Outdated Show resolved Hide resolved
@glepnir glepnir force-pushed the border_hl branch 3 times, most recently from b1a164c to 38c1b1a Compare May 12, 2024 06:07
@zeertzjq zeertzjq removed the needs:response waiting for reply from the author label May 12, 2024
@glepnir glepnir requested a review from bfredl May 18, 2024 13:07
src/nvim/drawscreen.c Outdated Show resolved Hide resolved
@bfredl
Copy link
Member

bfredl commented May 25, 2024

Togheter with this change, we could get rid of default_hl_id in parse_bordertext() and use -1 there, right? Would be a deduplication of logic.

Problem: there is missing default title highlight when highlight not defined in title text chunk.

Solution: when attr is not set use default title highlight group.
@bfredl bfredl merged commit 75f6ee5 into neovim:master May 25, 2024
29 checks passed
@glepnir glepnir deleted the border_hl branch May 25, 2024 09:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking-change float floating windows
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants