The only "problem" I see is that by moving the data-access controls and its events to the data-module you no longer have access by default to the form's controls, but that can be easily remedied by adding the form's unit to the uses clause in the implementation section of the data module and qualifying the accesses you make to the form's controls.
You might, perhaps, make it easier by adding some auxiliary methods to the form like, say: function GetClientsGrid: TDBGrid or whatever, to be able to work through the class rather than the instance, but that's usuallly just gilding that can be added as needed.
Depending on your application you'll have a little or a lot of work to do and some re-thinking time to dedicate, but that's common to almost any refactoring.
My recomendation, in short, is to move the event handlers to the same unit where its class resides: which in this case, for AfterScroll is the data module