| Commit message (Collapse) | Author | Age |
| | |
|
| |
|
| |
Apply "redundant cast to the same type" fix from clangd.
|
| | |
|
| | |
|
| |
|
|
| |
There's already an ignore directive but it's placed on the wrong line.
This commit should correct it.
|
| |
|
|
|
| |
False positive. Documentation for grid_scroll says "`cols` is always
zero, reserved for future use".
|
| | |
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
| |
* Revert "tui: improve support for tmux, GNU Screen"
This reverts commit c266c2f36ea56220268d2b19364e28815e338c2f.
This commit broke neovim under tmux, where it "types" the background
string as input to the terminal. Fixing this with neovim as-is requires
changing tmux keybindings or TERM settings, which has to be done by
every single user, therefore a fix should happen on neovim's side.
Fixes #14298
* tui: fix code lints.
|
| |
|
|
| |
Closes #14573
|
| |
|
|
| |
Wrapping it in DCS allows the get_bg features to work with tmux, GNU Screen.
|
| | |
|
| |
|
|
|
|
|
|
| |
assert() is compiled out for release builds, but we don't want to
continue running in these impossible situations.
This also resolves the "implicit fallthrough" warnings for the asserts
in switch cases.
|
| |
|
|
|
| |
Adjust the test for handle_background_color() according to
bd0275182b1c1b14c43dc4fc7e9f9da05071e56c.
|
| |
|
|
|
|
|
|
| |
Terminal responses may be fragmented. In that case, the problem that was
not processed normally and was processed in the same way as user input
is corrected.
fixes #11393.
|
| |
|
|
| |
fixes #11699, #11991.
|
| |
|
|
|
|
|
|
| |
termkey_strfkey() formats ctrl-l and ctrl-shift-l as <C-l> and <C-L>,
respectively. Nvim wants the latter to look like <C-S-L>, since <C-l>
and <C-L> are interpreted the same way.
This is only required when the Ctrl modifier is present.
|
| |
|
|
|
|
|
|
| |
This allows us to remove special-case handling of Esc in
forward_modified_utf8(), which was always sending "<Esc>" to nvim even
when there were modifiers present.
Closes #12584
|
| |
|
|
| |
fixes #12322.
|
| |
|
|
| |
fixes #13073.
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
- TUI: Fix a case where the cursor was not displayed after hiding the
cursor and then setting it to be displayed again.
- Change to reset everything before setting guicursor.
fixes #12800
close #12811
Steps to reproduce:
nvim -u NORC
:set termguicolors
:hi nCursor guifg=red guibg=red
:hi iCursor guifg=green guibg=green
:hi cCursor guifg=blue guibg=blue
:set guicursor=n:block-nCursor,i:hor25-iCursor,c:ver25-cCursor
:set guicursor-=c:ver25-cCursor
Actual behaviour: Cursor is a blue vertical.
Expected behaviour: Cursor should be the default color block.
|
| | |
|
| |
|
|
|
|
|
| |
This reverts commit f3ffe0b325170dd214b80e371bee5a56b7054940.
Sending a control sequence to the host terminal, wrapped in DCS, causes
problems because it changes the state of the host terminal outside the
screen's control. This change will therefore be reverted.
|
| |
|
|
| |
This reverts commit 9d0222ee3e232be859c3c70e6681f7156b5ac9d5.
|
| | |
|
| |
|
| |
It is unused since 8898793ad.
|
| | |
|
| |
|
|
|
|
| |
Tmux uses smso to display italics when the default-terminal is screen,
screen-*. That's not the display the user wants, so change it to do the
same display as before f3ffe0b32.
fixes #12193.
|
| |
|
|
|
|
|
|
| |
Wrapping it in DCS allows the following features to work with GNU
Screen.
- Changing the cursor style.
- bracketed paste.
- focus reporting.
|
| |
|
| |
fix #8075
|
| |
|
| |
Fixes #11653, #11990. Check for cursor visibility to toggle uv_write call.
|
| |\
| |
| | |
Closes #11998
|
| | | |
|
| | |
| |
| |
| |
| | |
gnu screen does not have smglr, but it inherits smglr from xterm and
splitting will cause drawing problems. So disable smglr.
|
| | | |
|
| |/
|
|
|
|
| |
Since "nvim_get_option" is executed on the tui thread as a C function
instead of msgpack-rpc, it accesses global variables that may change on the
main thread.
|
| |
|
| |
fixes #11893
|
| |
|
|
|
|
|
| |
Problem: Output of :command is hard to read.
Solution: Make some columns wider, some narrower. Truncate the command when
listing all.
https://github.com/vim/vim/commit/725310d89e1ba268bf410472b7de054c6c260161
|
| | |
|
| | |
|
| |
|
| |
when setting 'guicursor' highlight blend=100.
|
| |
|
|
| |
Ref: https://github.com/neovim/neovim/commit/ab7da4c53138768#r36200515
|
| | |
|
| | |
|
| | |
|
| |
|
| |
unibi_from_term calls getenv internally, so exclusive control is required.
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
`tui_terminal_after_startup` gets called right after resuming from
suspending (via `Ctrl-z`) already (not delayed as with the startup
itself), and would set `waiting_for_bg_response` to false then directly.
This results in the terminal response not being processed then anymore,
and leaking into Neovim itself.
This changes it to try 5 times always, which means that it typically
would stop after a few characters of input from the user typically, e.g.
with tmux, which does not send a reply.
While it might be better to have something based on the time (e.g. only
wait for max 1s), this appears to be easier to do.
Fixes regression in 8a4ae3d.
|
| |
|
|
|
|
|
|
|
| |
"smglr" was added for TERM=xterm recently to the terminfo database,
which causes display issues with terminals that use `TERM=xterm` by
default for themselves, although not supporting it.
This patch makes "smglr" to be ignored then.
Fixes https://github.com/neovim/neovim/issues/10562
|
| |
|
|
|
|
|
|
| |
* handle_background_color: short-circuit if handled already
* Unit tests for handle_background_color
* set waiting_for_bg_response to false in tui_terminal_after_startup
By then it should have been received.
|
| |
|
|
|
|
|
|
| |
Initially done in 3626d2107 (#11074, for #11062), it was reverted then
in 445f2f409 (#11083, which added flushing later).
But it is still required here to avoid the reporting of the background
response with urxvt/kitty (`nvim -u NONE -cq`).
Apparently I've tested this not enough with 445f2f409 (probably only within tmux).
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
| |
This avoids having a dummy event to tickle the main loop.
Confirmed using `nvim -u NONE -c 'au FocusGained * q'` in tmux (with
`:set -g focus-events on`): without the flushing it would only exit
after pressing a key.
Moves the flushing done recently in 3626d2107.
`nvim -u NONE -cq` is still working (i.e. consuming the response for the
terminal background query itself), and the flickering mentioned in
3626d2107 is reduced again.
Reverts part of bfb21f3e0 (#7729).
|