Hi Wei,

The event sequence is described in the documentation, and is the same between using the keyboard and the mouse: Grid InCell editing - Notes - Evets Sequence.

Using OnEdit to change the grid state is not going to solve the case where you want to pass a custom model instance to the grid. Moreover, OnEdit fires after the grid has tried making a copy of the row model. The event will fire once per row, you can find more details in the article linked above. We cannot guarantee changing the edited item through the grid state in OnEdit will not cause issues, and doing so does not remove the need for a parameterless constructor and will not remove the logic the grid executes.

If you have questions on the timeframes of the items we're working on with you, feel free to post them in any of the private threads you have about them.

Regards,
Marin Bratanov
Progress Telerik

Hello Wei,

As we have discussed a few times in your private ticket, OnEdit is not and will not be the place to change the model instance. OnNeedModel (or however the event gets called) will be the place to provide a custom implementation of the model, for cases when you cannot provide a parameterless constructor. A model instance is also needed for filtering, not only for inserting or editing rows.

This bug will fix the issue where pressing Enter on a row in InCell edit mode opens the next row for editing, but does not fire the OnEdit event for it. The target we have for this fix is the 2.24.0 release, which is the next release after the current one (2.23.0).

This bug, and the enhancement that will let you can provide custom model implementation (rather than the grid automatically creating one for you) are two completely separate tasks and they may not be completed in the same release.

Regards,
Marin Bratanov
Progress Telerik