Added OnClick and OnMouse actions for Canvas

[KristofferStrube]

As described in issue #83

[KristofferStrube]

I accidentally continued working with other things on the same branch. That has been reverted now. I'm new to making Pull Requests.

[ummerland]

Hi, perfect addition to the Canvas!
There are some other Events that can also be added however: OnPointerDown, OnPointerUp, OnPointerMove

[Thaina]

Did ondblclick was included?

[KristofferStrube]

Hey @ummerland I must have missed a notification at some point, sorry. But I added the handlers you described now. And added the handler that @Thaina requested as well together with the OnContextMenu handler.

[ummerland]

Very nice, thanks @KristofferStrube

[ummerland]

Very good and practical addition, clean code

[Thaina]

@ummerland Do we have any chance this would be updated in nuget? Or we need to build this ourselves? It seem this repo don't get any updated for a whole year

[attilah]

Sorry for the late update, we coming to this package and get review the feature PR and also update the package to .NET 5.0, thanks for the contrib.

[KristofferStrube]

@attilah regarding the conflict with blazor.extensions.canvas.js which is auto-generated. Which version should be used? Doesn't seem like it would actually need to change with the modifications I've made to the project.

[KristofferStrube]

@tbm0115 I had some problems with my merge as you can see, but there are no conflicts now. You can just squash my commits.

[neilgd]

Hi,

I'm wondering if this change made it into the release, or maybe it got lost somewhere?

I get an error System.InvalidOperationException: Object of type 'Blazor.Extensions.Canvas.BECanvas' does not have a property matching the name 'onmousedown'

[KristofferStrube]

Hey @neilgd,
No this never got merged into the master branch.

[sethfduke]

Hey @neilgd, No this never got merged into the master branch.

Any chance this is going to be merged at some point? Been sitting here for quite a while.