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

Adding double click functionality #319

Merged
merged 4 commits into from
Dec 8, 2015
Merged

Adding double click functionality #319

merged 4 commits into from
Dec 8, 2015

Conversation

maxwellfundi
Copy link
Contributor

I added the double click functionality for the selector user control.

Private Sub lstAvailableVariable_DoubleClick(sender As Object, e As EventArgs) Handles lstAvailableVariable.DoubleClick
CurrentReceiver.AddSelected()
CurrentReceiver.Focus()
End Sub
Copy link
Member

Choose a reason for hiding this comment

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

Good attempt but you are repeating code. This should call the add sub.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Okay, let me look at it again

@maxwellfundi
Copy link
Contributor Author

Here is the corrected version of the double click functionality for the data selector

volloholic added a commit that referenced this pull request Dec 8, 2015
@volloholic volloholic merged commit bb79ac2 into IDEMSInternational:master Dec 8, 2015
@maxwellfundi maxwellfundi deleted the AddingDoubleClickFunctionality branch December 15, 2015 07:23
dannyparsons added a commit that referenced this pull request Sep 14, 2016
stevenndungu referenced this pull request in stevenndungu/R-Instat May 1, 2017
dannyparsons pushed a commit that referenced this pull request May 3, 2017
maxwellfundi added a commit that referenced this pull request Jun 1, 2018
dannyparsons pushed a commit that referenced this pull request Aug 27, 2018
lloyddewit pushed a commit that referenced this pull request Apr 13, 2022
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.

2 participants