add mouseUp and mouseDown

[stephenkingsley]

I think add mouseUp and mouseDown event is convenient for this project.

[mjackson]

Why do you think this change is necessary? Can't you just add these events directly to your child component?

<PointTarget ...>
  <div onMouseUp={...}/>
</PointTarget>

[stephenkingsley]

because in the PointTarget had click event, so mouseUp or down is a part of click event.

[mjackson]

@stephenkingsley We're just cloning whatever element you give us and passing in an onClick prop. If you want to use onMouseUp or onMouseDown you can do so using the example above.

[stephenkingsley]

may I ask you what is the purpose of creating PointTarget. just the click event?

[battaile]

@stephenkingsley #2

[stephenkingsley]

maybe I knew what is the purpose of this component. and I agree it that the component is not need other event. and if I want to join your group, has any problem?