This fix tooltip text align

[mnsrv]

What do you think about fix behavior of ctx.fillText for align: 'center' or align: 'right' ? It's not working as expected.
I tried to solve this problem: change x coordinate depending on the alignment at Title, Body and Footer.

This solves #1977

[coveralls]

Coverage decreased (-0.2%) to 71.645% when pulling 6728d33 on sashamjolnir:patch-1 into b50c281 on nnnick:v2.0-dev.

[etimberg]

Hmm, I'm not sure about this. I agree that it eliminates the issue, but it also makes the alignment options useless.

The alignment options, if i remember correctly, were intended to allow customization of how the tooltip behaved around points. There were (they still might exist) some internal alignment strings and these options were supposed to set those.

[mnsrv]

According to documentation, tooltip text alignment options (*Align, *-title,body,footer) are only for text inside tooltip. I think this pull request doesn't changes tooltip position or tooltip behavior.

May be we should try to create new options to text alignment inside tooltip, because sometimes sometimes it's necessary and with this option we cannot achieve it (issue pics)

[etimberg]

You're correct that this is meant for the text. The issue I have is that by simply fitting the text no matter the alignment, the alignment options don't do anything because the text will always draw in the same place. My thought was that the text would be shifted by the alignment change, but that the tooltip would follow the text.

I think you are right that we need different options though.

[mnsrv]

Em, by the way, I didn't found some alignment options, that rules position of tooltip (top, bottom, left or right) relative to point (at line chart). You mean this title, body, footer Alignment option used to use for that?

[etimberg]

I had meant this stuff: https://github.com/nnnick/Chart.js/blob/master/src/core/core.tooltip.js#L339-L396

[panzarino]

Was this issue resolved in #2416?

[mnsrv]

@zachpanz88 no.
What I want is to align text inside tooltip. Because tooltip option bodyAlign: 'center', or titleAlign: 'center', is working not as I expect. See MDN Canvas

Examples

titleAlign: left;
bodyAlign: left

titleAlign: left;
bodyAlign: center

titleAlign: center;
bodyAlign: left

titleAlign: center;
bodyAlign: center

Thank you for responses and this great library.

[panzarino]

Closing because out of date

[mnsrv]

so this bug #1977 is still unresolved.

[panzarino]

We could be open to accepting a PR if it was based off of the current code in the master branch