[vim/vim] Correctly handle precedes char in list mode for long lines (#4953)
Christian Brabandt
The 'precedes' value for the listchars option should only be drawn when
the list option is on and wrapping is off (and there is text preceeding
the first column) OR when list option is on and wrapping is on and the
line is longer than the complete window (and there is still text
preceeding the first column). But in the latter case, only
draw the precedes value in the first column visible for the physical
line, not in every first column of the screen line.
Add a test to verify the behaviour.
related: neovim/neovim#11034
(Note: the change to view_util is actually unrelated, but makes debugging
easier as the window size does not change unexpectedly when trying to
run the test interactively and switching between windows).
You can view, comment on, or merge this pull request online at:
https://github.com/vim/vim/pull/4953
Commit Summary
- Correctly handle precedes char in list mode for long lines
File Changes
- M runtime/doc/options.txt (6)
- M src/screen.c (4)
- M src/testdir/test_display.vim (55)
- M src/testdir/view_util.vim (1)
Patch Links:
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
Justin M. Keyes
Zach Wegner (neovim/neovim#11034) should have been credited.
Christian Brabandt
> Am 22.09.2019 um 05:13 schrieb Justin M. Keyes <notifi...@github.com>:
>
> Zach Wegner (neovim/neovim#11034) should have been credited.
>
> —
> Reply to this email directly, view it on GitHub, or mute the thread.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.
Wang Shidong
@brammool This PR is merged without included origin patch author Zach Wegner?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.
Christian Brabandt
Am 22.02.2020 um 14:05 schrieb Wang Shidong (Vim Github Repository) <vim-dev...@256bit.org>:
@brammool This PR is merged without included origin patch author Zach Wegner?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.
--
--
You received this message from the "vim_dev" maillist.
Do not top-post! Type your reply below the text you are replying to.
For more information, visit http://www.vim.org/maillist.php
---
You received this message because you are subscribed to the Google Groups "vim_dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email to vim_dev+u...@googlegroups.com.
To view this discussion on the web visit https://groups.google.com/d/msgid/vim_dev/vim/vim/pull/4953/c589954433%40github.com.
Christian Brabandt
>
>
> @brammool This PR is merged without included origin patch author Zach Wegner?
>
> —
> Reply to this email directly, view it on GitHub, or unsubscribe.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.
Bram Moolenaar
It is mentioned in doc/version8.txt
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.