Support soft-wrapping of ghost text

[kevinushey]

Describe the feature

Currently, when displaying ghost text within an Ace instance, that text is not soft-wrapped. For example observe how the first line (real text) is wrapped, while the second line is not:

Use Case

Soft-wrapping of ghost text when enabled would better match the user intention.

Proposed Solution

It looks like ghost text is added as a single token "ghost_text" here:

ace/src/virtual_renderer.js

Lines 1761 to 1762 in 099011b

	var textLines = text.split("\n");
	this.addToken(textLines[0], "ghost_text", insertPosition.row, insertPosition.column);

I wonder if the ghost text could be split into separate tokens, and added individually? A simple implementation would be to just split on whitespace, and I think that would be enough to make sure the rendered text obeys soft-wrapping rules.

Other Information

No response

Acknowledgements

• I may be able to implement this feature request
• This feature might incur a breaking change

ACE version used

version 1.32.7

[akoreman]

We added soft-wrapping of ghost text as part of #5540, it will be included in the next Ace release after 1.33.1.

[kevinushey]

That's fantastic -- thank you!