Fix an issue while calling onTap: close #8 - Githubissues

fluttercandies / flutter-interactive-chart

A candlestick chart that supports pinch-to-zoom and panning.

https://pub.dev/packages/interactive_chart

MIT License

209 stars 66 forks source link

Fix an issue while calling onTap: close #8 #10

Closed kcoms555 closed 2 years ago

kcoms555 commented 2 years ago

Summary

8 onTap is not working as described

Problems

https://github.com/fluttercandies/flutter-interactive-chart/blob/8bce5d633379995685b4a2351d8a5f9f81955997/lib/src/interactive_chart.dart#L214-L218

_fireOnTapEvent() needs position information to get a position of a candle.
But assignment of null on _tapPosition inside setState() make it unavailable.

Expected Results

make onTap() work.