bugfix: fix CommandOnEnterBehavior not detecting 'enter' key

[benthecarman]

Addresses some comments in #1450

[danwalmsley]

@benthecarman thanks for this.

Most applications seem to execute commands on KeyDown rather than KeyUp.

Fear not however, the cause of the original bug is in the same area. Something else is handling the keydown event before it has chance to get to you.

You can resolve this by passing RoutingStrategies.Tunnel to the routes parameter or AddHandler as per.

https://github.com/AvaloniaUI/Avalonia/blob/master/src/Avalonia.Interactivity/Interactive.cs#L42

if that doesnt work, then try setting the handledEventsToo parameter to true (as a last resort).

Let me know if works and if you agree this would be a better fix, and if you want to know more about what all this Tunneling and Bubbling is just reach out to me here.

👍

[danwalmsley]

I think it might because some other control in the UI tree is probably setting the handled flag to true when the enter key is pressed. This means that your event handler wouldnt get called.

[benthecarman]

Your fix worked, thanks!

[danwalmsley]

@benthecarman awesome, thanks for the contribution 👍

[danwalmsley]

Looks good to me.