Conversation
4525f76 to
32d75e4
Compare
af717ea to
417d677
Compare
|
I found #11718 which this PR is related to, seems like bram did not get the point of something like this. However it has noticeably reduces flickering for me, so I guess there is a point. I think this is ready |
d589505 to
af7f30f
Compare
Hm, we typically have those as |
|
This PR fixes the popup flickering, but does it as a side effect of the feature, also since it only does it for terminal that supports this feature, I think that having correctness back for popups without transparencies is important. So opaque popups don't require terminals with this feature to avoid rendering issues, while popups with transparencies will require it for avoiding flickering. Due to this, I think that even with this PR, the with changes from #19534 are still required. Said this, I think it's a great feature to have 👍 . |
The synchronized output does not use termcap to check if it is available, so I don't think we need a |
41d6c3c to
a089f84
Compare
|
Okay I have made it so the escape codes to begin and end synchronized output are configurable by the user. I left the |
|
Does this the same as in Neovim? If yes, I assume we cannot change the naming? Because if I hear |
I named it |
|
let's keep it the same as in Neovim |
Yes. Since it is essentially a control for the terminal, I think the |
|
I think this is ready |
|
thanks |
related: #19541 Signed-off-by: Christian Brabandt <cb@256bit.org>
Same as Neovim's
termsyncoption. Surprised this wasn't already added, considering how simple it was to implement (unless I'm missing something) and it reduces flickering greatly for me on my machine.Here's a before and after comparsion:
2026-03-01.13-31-14.mp4