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(echospace): don't subtract SHOWCMD_COLS if showcmdloc is statusline or tabline #14108

Closed
wants to merge 5 commits into from
Closed

Conversation

Sam-programs
Copy link
Contributor

The command is not visible in the message area when showcmdloc is statusline making echospace smaller than what it can be.

src/screen.c Outdated Show resolved Hide resolved
Co-authored-by: zeertzjq <zeertzjq@outlook.com>
@zeertzjq
Copy link
Member

Hmm, it seems that a comp_col() is needed in did_set_showcmdloc().

@Sam-programs
Copy link
Contributor Author

Sorry for the delay, i had internet problems.

@chrisbra
Copy link
Member

thank you!

@chrisbra chrisbra closed this in 062141b Feb 29, 2024
zeertzjq added a commit to zeertzjq/neovim that referenced this pull request Feb 29, 2024
Problem:  the amount of available space (v:echospace) on the command
          line is not correct when showcmdloc is drawn into the
          statusline or tabline.
Solution: only add SHOWCMD_COLS to the shown command column when
          'showcmdloc' is set to last (Sam-programs)

closes: vim/vim#14108

vim/vim@062141b

Co-authored-by: Sam-programs <130783534+Sam-programs@users.noreply.github.com>
zeertzjq added a commit to neovim/neovim that referenced this pull request Feb 29, 2024
…#27682)

Problem:  the amount of available space (v:echospace) on the command
          line is not correct when showcmdloc is drawn into the
          statusline or tabline.
Solution: only add SHOWCMD_COLS to the shown command column when
          'showcmdloc' is set to last (Sam-programs)

closes: vim/vim#14108

vim/vim@062141b

Co-authored-by: Sam-programs <130783534+Sam-programs@users.noreply.github.com>
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.

None yet

3 participants