Release resources in $on('$destroy') - fixes major memory leak

fusioncharts / angularjs-fusioncharts

AngularJS Directive for FusionCharts JavaScript Charting Library (for Angular 1.x)

MIT License

111 stars 101 forks source link

Closed ScallyGames closed 6 years ago

ScallyGames commented 8 years ago

Free FusionChart resources on directive destruction. Fixes memory leakage (see attached images).

Before: beforefix After: afterfix

PS: There still seems to be some memory leak

ScallyGames commented 8 years ago

Any updates on this? Would really love to switch back to the official repo.

bensgroi commented 7 years ago

Looks like PRs aren't being merged anymore, but thanks for submitting this!

rousan commented 6 years ago

Hi @Aides359, We are working on this memory leak issue and will be available in next patch release soon, and we should close this PR.