helix icon indicating copy to clipboard operation
helix copied to clipboard

Soft wrap

Open kirawi opened this issue 2 years ago • 3 comments

Objective

This PR will resolve #136. On Matrix, @cessen suggested splitting the overall soft-wrapping feature into two PRs for each of the two techniques he proposed. This PR will only implement the first, which should be enough for the majority of cases.

Status

The View position is not yet represented by a char offset into the text, and so editing is not yet feasible. Rendering is also partial, as it will not correctly calculate the lines, won't wrap on a word break, nor will the cursor wrap to the next line once it goes past the screen. While I am reasonably confident about how to approach implementing this, it's being delayed by a lack of time on my part.

Will depend upon #3268

kirawi avatar Apr 20 '22 01:04 kirawi

Hey there @kirawi, now that Markdown is supported out-of-the-box, I think it'd be great to be able to use soft wrap, seeing as we might be writing paragraphs and whatnot. I see you've merged a few changes since your last update to the PR, so just wondering what's missing or if editing is feasible yet? Thank you for your work on this!

jfaz1 avatar Sep 19 '22 04:09 jfaz1

It's far from ready. I'm waiting for motivation to complete #3268 which entails implementing a cached line wrapper iterator.

kirawi avatar Sep 20 '22 22:09 kirawi

I'm waiting for motivation to complete #3268...

Maybe we can send cookies.

theherk avatar Oct 19 '22 16:10 theherk

Closing in favor of #5008

kirawi avatar Dec 05 '22 00:12 kirawi