Justified text sometimes expands past maxWidth

[simonbengtsson]

Sometimes when drawing text with the justify alignment option set the text expands past maxWidth. Red lines in the below example highlights the maxWidth length and the fillerrrrr word can be seen expanding past it.

The same example works when using left alignment:

Here is the code for the example:

var doc = new jsPDF();

// maxWidth lines
doc.setDrawColor(255, 0, 0);
doc.setLineWidth(0.5);
doc.line(10, 10, 10, 33);
doc.line(60, 10, 60, 33);

doc.text(10, 20, 'filler shortword longerword fillerrrrrr', {
    maxWidth: 50,
    align: 'justify'
});

doc.save('table.pdf');

[bwl21]

this seems to be a bug.

You can reproduce and playaround this in Live Demo.

if you add a space to the string, it works well.

var doc = new jsPDF();

// maxWidth lines
doc.setDrawColor(255, 0, 0);
doc.setLineWidth(0.5);
doc.line(10, 10, 10, 33);
doc.line(60, 10, 60, 33);

doc.text(10, 20, 'filler shortword longerword fillerrrrrr ', {
    maxWidth: 50,
    align: 'justify'
});

[github-actions]

This issue is stale because it has been open 90 days with no activity. It will be closed soon. Please comment/reopen if this issue is still relevant.

[simonbengtsson]

Still an issue

[themacboy]

Probably this is issue is related to my issue about wrong calculations in justified text and use of gettextdimension function. El mié., 8 jul. 2020 7:32, Simon Bengtsson <[email protected]> escribió:

…

Still an issue — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#2553 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AB5QVX67NSL4P47YAEQSXR3R2QAINANCNFSM4IKTQ3DQ> .

[simonbengtsson]

This should not be closed?

[HackbrettXXX]

Interesting, I thought commenting would be enough such that the bot doesn't close the issue.

[github-actions]

This issue is stale because it has been open 90 days with no activity. It will be closed soon. Please comment/reopen if this issue is still relevant.

[simonbengtsson]

Still relevant

[SimonPrins]

@simonbengtsson @HackbrettXXX I believe this can be closed now. This is what the example looks like in the playaround:

I believe this PR fixed it: #3168
Which was released with the 2.4.0 release