✨ Indent wrapped lines

[ben-p-commits]

Is your feature request related to a problem? Please describe.

In the enhancement requested by CodeEditApp/CodeEdit#868, the request to add configurable indentation to wrapped lines is beyond the current capabilities of CodeEditTextView.

Describe the solution you'd like

Add a parameter or option which allows custom indentation to be added for wrapped lines. May require enhancement + PR in STTextView

Additional Context

Example behavior in Xcode:

4 spaces

12 spaces

[Bogdan-Belogurov]

@thecoolwinter Hi! Can I take this one?

[austincondiff]

@Bogdan-Belogurov Where are we with this? No rush, just checking.

[Bogdan-Belogurov]

@austincondiff @thecoolwinter Hi! I'm still in progress.
I'm trying to find the easiest way to implement this logic. I found this PR with TextLayoutManagerRenderDelegate, but the current TextView doesn't use this (only MinimapView). Am I right to add the realisation of the TextLayoutManagerRenderDelegate protocol to the TextView?

[thecoolwinter]

@austincondiff @thecoolwinter Hi! I'm still in progress.
Am I right to add the realisation of the TextLayoutManagerRenderDelegate protocol to the TextView?

Not quite, that protocol is to allow developers to override layout or rendering behavior. CETV handles that by default but in the minimap's case it overrides how text is drawn (little dots).

Three places to look:

• LineFragmentRenderer does the actual drawing of the text in a graphics context. This will need to be modified to draw the indented fragments correctly.
• Typesetter determines line breaks - you'll want to modify this one to allow less width for line fragments that should be indented, which will change where line breaks occur.
• LineFragment will need to know about any indentation and adjust to account for that for instance when determining the content offset of an X coordinate in the fragment.

[alechash]

Hey @Bogdan-Belogurov and @thecoolwinter, not sure of the progress on this but if it has stalled I can take it. Could commit a few hours over the next two days + more this upcoming week.

Thanks, let me know!

[vksvicky]

Is anyone working on this? I can pick it up and have a look!

[alechash]

@vksvicky I would say go for it. I never heard back from them when I asked and by now I don’t have much free time at the moment to put anything toward it.

[vksvicky]

@alechash please assign this to me, so I can change the status and add relevant notes as I progress