Am 03.04.2018 um 14:41 schrieb Maxim Ganetsky via Lazarus:
I reopened this bug report. Please comment there.


But since other opinions are required we should discuss here - the bug tracker is not the place for discussion. I applied the change leading to this issue would like to hear other opinions. Here's what I wrote in bug tracker:

"I don't see anything wrong in the logics of the new behavior: The OnChanging event is fired before the selected node is changed. It has the node which will become the new selected node as a parameter, and you can do some validation and signal in AllowChange whether the change to the new node will be allowed or not. In the event handler you still have access to the currently selected node as TreeView.Selected. Therefore, there is no need at all to duplicate the currently selected node in the node parameter of the event.

This behavior is now compatible with Delphi (it was not before the fix).

Like any bugfix the new code breaks backward compatibility - sorry for the inconvenience if you have relied on the old behavior.

Which other controls have the OnChanging event? I only found TPageControl/TTabControl but they don't pass the changing tab as a parameter. "

--
_______________________________________________
Lazarus mailing list
Lazarus@lists.lazarus-ide.org
https://lists.lazarus-ide.org/listinfo/lazarus

Reply via email to