EventEmitter type is object, should be ActiveElement

[florestankorp]

Reproduction of the problem

<canvas (chartClick)="chartClicked($event)"> </canvas>

public chartClicked(event: { event?: ChartEvent; active?: object[] }): void {
  if (event.active && event.active.length > 0) {
    const myChartDataLabelIndex = event.active[0].index; // `Property 'index' does not exist on type 'object'.ts(2339)`
    console.log(myChartDataLabelIndex);
  }
}

Suggested fix:

libs/ng2-charts/src/lib/base-chart.directive.ts

  @Output() public chartClick: EventEmitter<{
    event?: ChartEvent;
    active?: ActiveElement[];
  }> = new EventEmitter();

  @Output() public chartHover: EventEmitter<{
    event: ChartEvent;
    active: ActiveElement[];
  }> = new EventEmitter();

[Matthiasvanderhallen]

We would love to see this fixed, I looked around in the codebase and found no reason for active to be typed as the less restrictive object[]... Nor for the possibility of leaving those undefined either.

To be honest, even better would be a

type ChartMouseEvent = { event:ChartEvent, active: ActiveElement[] }