bugfix/14143-14434-stickOnContact-useHTML-outside

[khlieng]

Fixed #14143, #14434, #13310, tooltip with stickOnContact and useHTML or outside enabled did not work properly.

Replaces #14473.
Closes #14434.
Closes #13310.

[highsoft-bot]

File size comparison

	master	candidate	difference
highcharts	290.5 kB	290.4 kB	-104 B
highcharts, gzipped	97.7 kB	97.6 kB	-201 B
highstock	386.6 kB	386.5 kB	-104 B
highstock, gzipped	128.7 kB	128.4 kB	-300 B
highmaps	349.0 kB	348.9 kB	-110 B
highmaps, gzipped	114.6 kB	114.5 kB	-30 B
highcharts-gantt	398.1 kB	398.0 kB	-106 B
highcharts-gantt, gzipped	132.5 kB	132.3 kB	-186 B

[highsoft-bot]

Visual test results - No difference found

[TorsteinHonsi]

Thanks!

[galangel]

I'm sorry, but it still an issue:

    useHTML: true,
    stickOnContact: true,
    outside: true,

are not working together as expected

for example, here: https://jsfiddle.net/gh/get/library/pure/highcharts/highcharts/tree/master/samples/highcharts/demo/accessible-line

adding outside: true, brakes stickOnContact

[doganalper]

This is still ongoing issue, can't use stickOnContact with outside configuration.

[khlieng]

Worked from 9.2.0 to 10.2.1, broken again since 10.3.0, which makes #17781 a likely suspect 🧐 @TorsteinHonsi

[TorsteinHonsi]

Suspicion confirmed. Bisected to 2e5bdd7.

[TorsteinHonsi]

@doganalper I'll have a look at this ASAP

[TorsteinHonsi]

@doganalper , I found a fix, created a PR.

Here's a drop-in workaround that you can use while waiting for the next release:

Highcharts.Pointer.prototype.onContainerMouseLeave = function(e) {

    const { charts, pick, Pointer } = Highcharts;
    const chart = charts[pick(Pointer.hoverChartIndex, -1)];

    e = this.normalize(e);

    // #4886, MS Touch end fires mouseleave but with no related target
    if (
        chart &&
        e.relatedTarget &&
        !this.inClass(e.relatedTarget, 'highcharts-tooltip')
    ) {
        chart.pointer.reset();
        // Also reset the chart position, used in #149 fix
        chart.pointer.chartPosition = void 0;
    }
}

View it live on jsFiddle.

[doganalper]

@TorsteinHonsi Workaround works perfectly, thank you so much for helping!