breakTextIntoLines - line widths

[akomm]

Currently you need to compute the width of the lines twice in order to implement horizontal alignment of the text. Once its computed in breakTextIntoLines via look ahead adding words, then if you want to align the lines, you again need the width. You don't need to compute the width that many times here, but you still need to do it again.

The Idea:

type Line = {
  text: string
  width: number
}
function breakTextIntoLines(/* same api as now */): Line[]

Which is basically a simple change to the const lines structure and this few lines of code:

pdf-lib/src/utils/strings.ts

Lines 119 to 123 in 2047bb8

	const pushCurrLine = () => {
	if (currLine !== '') lines.push(currLine);
	currLine = '';
	currWidth = 0;
	};

However, this would be a BC break on the API. Hence, not so good. I would like to ask what you think about the idea in general and would you say its worth create another util function for that case and if so, how would you name it. Or did I miss some other way around the issue of getting the measure?

Sure, I could just compute the line width again...