Add keep_newlines argument to wrap_text

[theFox6]

If keep_newlines is true, every new line given in the input text is also contained in the output text.

[theFox6]

Well this looks more clean than #7232

[theFox6]

Can this be added please?

[SmallJoker]

Testing mod:

local texts = {
--                                               | 50 is here
[[

Lorem ipsum dolor sit amet, consetetur sadipscing elitr, sed diam nonumy]],

[[eirmod tempor invidunt ut labore et dolore
magna aliquyam erat, sed diam voluptua. At vero eos et accusam et justo duo
dolores et ea rebum. Stet clita]]
}

print("core.line_break(text, 50)")
for i, text in ipairs(texts) do
	local lines = core.line_break(text, 50)
	print("\t" .. table.concat(lines, "\n\t"))
	print("")
end
print("core.wrap_text(text, 50, true, newlines)")
for i, text in ipairs(texts) do
	local lines = core.wrap_text(text, 50, true, true)
	print("\t" .. table.concat(lines, "\t")) -- new lines contained
	print("")
end

print("core.wrap_text(text, 50, true, no newlines)")
for i, text in ipairs(texts) do
	local lines = core.wrap_text(text, 50, true, false)
	print("\t" .. table.concat(lines, "\n\t"))
	print("")
end

But it does not contain new lines? This is the output:

core.wrap_text(text, 50, true, newlines)
		Lorem ipsum dolor sit amet, consetetur sadipscing	elitr, sed diam nonumy	

	eirmod tempor invidunt ut labore et dolore		magna aliquyam erat, sed diam voluptua. At vero	eos et accusam et justo duo		dolores et ea rebum. Stet clita	

[theFox6]

The previously contained newlines were converted into table splits, I've added a note for that to the documentation. I can change the behavior so it wraps into table lines and puts the newlines in as characters if you want that.

[rubenwardy]

It would be great to have some unit tests for this, see the tests folders in builtin

Also, this is yet another wrapping function in Minetest that #10853 could replace :'(

[SmallJoker]

https://irc.minetest.net/minetest-dev/2022-07-03#i_5990806

Concept approved.

[Zughy]

Hello @theFox6 , sorry for the delay. Now that the PR has been conceptually approved, do you think you can take care of the rebase?