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

fix typo: event reference #14

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

daithi-coombes
Copy link

Fix typo referencing the event object in SuperCalendar.onDayClick

issue: #13

@@ -9,7 +9,7 @@ SuperCalendar = {
});
},
onDayClick: function (e, t, data) {
var target = event;
var target = e;
Copy link
Collaborator

Choose a reason for hiding this comment

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

'target' is defined but never used.

@gabrielhpugliese
Copy link
Owner

Thanks!
Maybe we can remove it since it is not being used.

@gabrielhpugliese
Copy link
Owner

Hmmmm I think I can't change your PR because it's a fork. Would you mind doing it?

@daithi-coombes
Copy link
Author

sorry for the late reply. Am on it.

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.

3 participants