TChartAxis.OnMarkToText should include axis

[piola]

I'd like to suggest the following change:

TChartAxisMarkToTextEvent = procedure (Sender: TObject; var AText: String; AMark: Double) of object;

i.e. the axis firing this event should include a self-reference.

Reason: When dealing with multiple charts and/or multiple axes, it is currently required to create an individual handler for each axis because it is not possible to determine the axis for which to create axis marks for. An example would be an automatic adjustment of unit prefixes (milli, kilo, Mega, etc.) which would imply the same code for each axis, but would require to determine the min/max range of the axis. Currently, this ain't possible.

[wp]

Makes sense...

The problem is that a change in the parameters of a published event breaks existing projects using this event. The correct way to handle this situation is to deprecate the current event, add a new event with the new parameter combination and remove the old event in some future release.

OK, do you have an idea for the the name of the new event? I think the current name is perfect which makes it difficult to find a new name...

[piola]

How about "OnGetAxisMarkText"?

[wp]

See r62116, also https://wiki.freepascal.org/Lazarus_2.2.0_release_notes#TAChart:_TChartAxis.OnMarkToText_event_deprecated. Please test.

[piola]

Seems to work properly. Thank you very much!