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

complete arguments in addEventListener function #9273

Merged
merged 1 commit into from
Apr 5, 2021

Conversation

ahaWDY
Copy link
Contributor

@ahaWDY ahaWDY commented Mar 29, 2021

Signed-off-by: ahaWDY [email protected]

What it does

complete arguments of addEventListener function and fix #9250

How to test

The argument "useCapture" is used in the addEventListener function, so it should also add to the removeEventListener function

Review checklist

Reminder for reviewers

@vince-fugnitto vince-fugnitto added the quality issues related to code and application quality label Mar 29, 2021
Copy link
Member

@vince-fugnitto vince-fugnitto left a comment

Choose a reason for hiding this comment

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

@ahaWDY thank you for the contribution 👍
Unfortunately, in order to accept the changes the sign-off should be updated as the current one is invalid based on project guidelines. We do not accept anonymous contributions (ex: github-noreply).

Use your real name (sorry, no pseudonyms or anonymous contributions.)

Signed-off-by: ahaWDY <[email protected]>

complete arguments in addEventListener function
@ahaWDY
Copy link
Contributor Author

ahaWDY commented Mar 29, 2021

Use your real name (sorry, no pseudonyms or anonymous contributions.)

Thank you for your reply! I have updated the author and sign-off information.

Copy link
Member

@vince-fugnitto vince-fugnitto left a comment

Choose a reason for hiding this comment

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

Thank you for your contribution @ahaWDY 👍

@vince-fugnitto vince-fugnitto merged commit 3381872 into eclipse-theia:master Apr 5, 2021
@vince-fugnitto vince-fugnitto added this to the 1.13.0 milestone Apr 6, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
quality issues related to code and application quality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

the function of addEventListener has a bug
2 participants