Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update state of (mouse) botton in move handler #3781

Closed
wants to merge 8 commits into from
Closed

Update state of (mouse) botton in move handler #3781

wants to merge 8 commits into from

Conversation

rexrainbow
Copy link
Contributor

@rexrainbow rexrainbow commented Jun 23, 2018

This PR (delete as applicable)

  • Fixes a bug

Describe the changes below:

Update state of (mouse) botton in move handler. #3779

@rexrainbow
Copy link
Contributor Author

this.buttons has been updated in last commit. Please review it.

@rexrainbow
Copy link
Contributor Author

Sorry I add another PR about tween which is not related to issue of mouse event.
(BTW, How to submit multiple PRs?)

@rexrainbow
Copy link
Contributor Author

Here is a new PR, add Offset, OffsetPoint methods of polygon shape.

@photonstorm
Copy link
Collaborator

Hi - thanks for this PR, sorry it took a while to get around to it. I have fixed the Pointer.move issue and the Tween returns locally and pushed them to master (as too many changes had occurred since your PR to merge it safely).

I like the look of the two Polygon functions but could you please complete the jsdocs for them? I can't merge without them.

@rexrainbow rexrainbow closed this Jan 24, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants