Text right alignment not working with text array

[rcwwilliams]

Hi

Really appreciate how you are continuing to improve this great library.

Text right alignment works fine with a single string passed to doc.text, but does not seem to line up correctly with an array passed.

Quite easy to reproduce.

Thanks

Robin

[Uzlopak]

If it easy to reproduce you should be able to give us a code sample.

[rcwwilliams]

doc.text(200, 10, ['One', 'Two', 'Three'], null, null, 'right');

[Uzlopak]

I really dont know if I should laugh or cry. Do you have respect for your colleagues? Because you give me one line and expect from me to build a working code sample. Is my time less worth than yours? I dont think so.

Unless you provide a working code sample in jsfiddle or jsbin.com I am out.

[rcwwilliams]

Just paste that line in http://rawgit.com/MrRio/jsPDF/master/

that only takes a few seconds

[MrRio]

Yeah that looks like an easy bug for someone to pick up. Thanks for reporting.

On Sat, 3 Jun 2017 at 12:37, rcwwilliams ***@***.***> wrote: Just paste that line in http://rawgit.com/MrRio/jsPDF/master/ that only takes a few seconds — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#1310 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAC5s-w5qd61CNmB6B1e1adi8cNKTcrwks5sAUWJgaJpZM4Nua5-> .

-- James Hall Director Parallax Agency Ltd M. 07894950320

[Uzlopak]

@MrRio

for (var i = 1, len = da.length; i < len; i++) {
           var delta = maxLineLength - lineWidths[i];
           if (align === "center") delta /= 2;
           // T* = x-offset leading Td ( text )
           text += ") Tj\n" + (left - prevX + delta) + " -" + leading + " Td (" + da[i];
           prevX = left + delta;
         }

Seems to be a problem in that area....

[Kyderman]

Has this been looked at?

[Uzlopak]

Yeah. Code is not working. I fixed it in #1448.

[Kyderman]

Alright, I'll wait for the merge. Assuming that wouldnt take too long?

[Uzlopak]

I can't see it getting merged any time soon

[Uzlopak]

Hopefully we will merge fix it soon.

[Uzlopak]

Fixed in #1433
Will be part of next release