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

Enable reflow when supported #688

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from
Draft

Conversation

blahgeek
Copy link
Contributor

@blahgeek blahgeek commented Nov 5, 2023

term reflow is supported since libvterm v0.3. (the libvterm version in the cmakelists already includes this)

how to reproduce:

  1. echo a long line of text
  2. reduce the window width to be shorter than the line of the text
  3. increase the window width to the original size

old behavior: the text is truncated and not recovered

new behavior: the text is reflowed

similar to this change in neovim: neovim/neovim#21124

However, it seems that this would break per-line directory info while reflowing (I'm not very sure, I don't use that feature). Do you have any suggestions?

@blahgeek blahgeek marked this pull request as draft November 5, 2023 09:37
term reflow is supported since libvterm v0.3

1. echo a long line of text
2. reduce the window width to be shorter than the line of the text
3. increase the window width to the original size

old behavior: the text is truncated and not recovered

new behavior: the text is reflowed
@dcaixinha
Copy link

I'd love to get this in emacs-libvterm 🙏 Especially useful when temporarily maximizing windows back-and-forth.

I guess the block for getting this merged is this?

it seems that this would break per-line directory info while reflowing

Can you share more details on what this means @blahgeek?

@martinbaillie
Copy link

Would also love this feature. I reckon many folks would, they just don't know it yet :-)

@blahgeek
Copy link
Contributor Author

blahgeek commented Apr 5, 2024

Sorry folks, this PR is still a draft since I encountered several cases of random crashes while running with this patch and I haven't got the time to debug yet.

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.

3 participants