[vim/vim] Pmenu is not shown if t_Co=0 (Issue #20800)

[Maxim Kim]

habamax created an issue (vim/vim#20800)

Steps to reproduce

1. vim --clean
2. :set t_Co=0
3. insert

hello
help

4. type h<C-n>

Expected behaviour

Pmenu should be shown.

When t_Co is not a 0:
image.png (view on web)

Note, that with set wildoptions=pum popup menu is shown for the cmdline:

image.png (view on web)

Version of Vim

9.2.818

Environment

debian13, bash, alacritty

Logs and stack traces

—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications, keep track of coding agent tasks and review pull requests on the go with GitHub Mobile for iOS and Android. Download it today!
You are receiving this because you are subscribed to this thread.Message ID: <vim/vim/issues/20800@github.com>

[h_east]

h-east left a comment (vim/vim#20800)

The documentation states the following:

INSERT COMPLETION POPUP MENU                *ins-completion-menu*
(snip)...
The menu is used when:
(snip)...
- The terminal supports at least 8 colors.
(snip)...

However, currently, this restriction only applies to the insert mode completion menu.
Since the command-line completion menu does not have this limitation, it feels inconsistent from a specification standpoint.

I'll write a patch that removes this restriction and uses terminal settings that remain clear and legible.

—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications, keep track of coding agent tasks and review pull requests on the go with GitHub Mobile for iOS and Android. Download it today!

You are receiving this because you are subscribed to this thread.Message ID: <vim/vim/issues/20800/5031859090@github.com>

[Maxim Kim]

habamax left a comment (vim/vim#20800)

Thx, Pmenu should be handled in #20799 (comment)

—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications, keep track of coding agent tasks and review pull requests on the go with GitHub Mobile for iOS and Android. Download it today!

You are receiving this because you are subscribed to this thread.Message ID: <vim/vim/issues/20800/5031996491@github.com>

[h_east]

h-east left a comment (vim/vim#20800)

uses terminal settings that remain clear and legible.

This refers to the default highlighting settings within the Vim source code.

—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications, keep track of coding agent tasks and review pull requests on the go with GitHub Mobile for iOS and Android. Download it today!

You are receiving this because you are subscribed to this thread.Message ID: <vim/vim/issues/20800/5032406964@github.com>

[h_east]

h-east left a comment (vim/vim#20800)

Ah, I see. Wouldn't it be best to supplement the PR #20803 with the missing items via PR #20799?

—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications, keep track of coding agent tasks and review pull requests on the go with GitHub Mobile for iOS and Android. Download it today!

You are receiving this because you are subscribed to this thread.Message ID: <vim/vim/issues/20800/5032477307@github.com>

[Maxim Kim]

habamax left a comment (vim/vim#20800)

Whatever would be merged first. I don’t mind updating my PR if that would be needed

—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications, keep track of coding agent tasks and review pull requests on the go with GitHub Mobile for iOS and Android. Download it today!

You are receiving this because you are subscribed to this thread.Message ID: <vim/vim/issues/20800/5032523033@github.com>

[Maxim Kim]

habamax left a comment (vim/vim#20800)

Btw you might be missing pmenuextra and pmenukimd

—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications, keep track of coding agent tasks and review pull requests on the go with GitHub Mobile for iOS and Android. Download it today!

You are receiving this because you are subscribed to this thread.Message ID: <vim/vim/issues/20800/5032531957@github.com>

[Maxim Kim]

habamax left a comment (vim/vim#20800)

I have removed Pmenu term definitions from my PR.

—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications, keep track of coding agent tasks and review pull requests on the go with GitHub Mobile for iOS and Android. Download it today!

You are receiving this because you are subscribed to this thread.Message ID: <vim/vim/issues/20800/5034176414@github.com>

[h_east]

h-east left a comment (vim/vim#20800)

I have removed Pmenu term definitions from my PR.

Ah, I'm sorry for making you go out of your way.
BTW, is the following matter okay?
#20803 (comment)
#20803 (comment)

—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications, keep track of coding agent tasks and review pull requests on the go with GitHub Mobile for iOS and Android. Download it today!

You are receiving this because you are subscribed to this thread.Message ID: <vim/vim/issues/20800/5035877990@github.com>

[Maxim Kim]

habamax left a comment (vim/vim#20800)

BTW, is the following matter okay?

yeah, should be ok

—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications, keep track of coding agent tasks and review pull requests on the go with GitHub Mobile for iOS and Android. Download it today!

You are receiving this because you are subscribed to this thread.Message ID: <vim/vim/issues/20800/5040111527@github.com>