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

OnButtonClicked event is called twice on every tap #2525

Closed
wants to merge 1 commit into from

Conversation

ei2kpi
Copy link

@ei2kpi ei2kpi commented Jul 31, 2018

OnButtonClicked is called twice since DoButtonPressed is called OnInputDown and OnInputClicked. Splitting out Clicked into it's own DoButtonClicked function prevents this from happening.

Overview

Changes

OnButtonClicked is called twice since DoButtonPressed is called OnInputDown and OnInputClicked. Splitting out Clicked into it's own DoButtonClicked function prevents this from happening.
/// <summary>
/// Called when button is "Clicked".
/// </summary>
protected void DoButtonClicked(bool bRelease = false)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

virtual?

@StephenHodgson StephenHodgson changed the base branch from master to june18_dev July 31, 2018 15:30
@keveleigh
Copy link
Contributor

This is already fixed for the next release in #2367

@ei2kpi
Copy link
Author

ei2kpi commented Jul 31, 2018

Closing since this was already fixed for next release...

@ei2kpi ei2kpi closed this Jul 31, 2018
@ei2kpi ei2kpi deleted the patch-1 branch July 31, 2018 16:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants