Allow bound method as event handler

[benedikt-bartscher]

No description provided.

[adhami3310]

We have this behavior already in the code? That's the number_of_bound_args argument.

[adhami3310]

Also, in a case like this, tests would be appreciated to highlight if the existing code for this has regressed

[benedikt-bartscher]

We have this behavior already in the code? That's the number_of_bound_args argument.

Yes, but it is not always set correctly. For some reason, it needs to be passed to check_fn_match_arg_spec.

Also, in a case like this, tests would be appreciated to highlight if the existing code for this has regressed

i will add one

[benedikt-bartscher]

@adhami3310 I added a test which is similar to our regression. We are using smth like this for quite a while now and it works fine. However, this is not really a documented/supported feature of reflex. Hence, this PR could be seen as a feature, not a regression bug fix

[adhami3310]

Since it's similar to the lambda feature, I think this one makes sense. Although it would have been cooler to unite it with that other path.