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

Added delay if event type is keyup #62

Closed
wants to merge 3 commits into from
Closed

Added delay if event type is keyup #62

wants to merge 3 commits into from

Conversation

edgardmessias
Copy link
Contributor

No description provided.

namezero111111 added a commit to namezero111111/redmine_didyoumean that referenced this pull request Sep 26, 2014
@abahgat
Copy link
Owner

abahgat commented Oct 2, 2014

Thanks for taking care of this Edgar, the PR looks good, the only thing I'd ask is to make sure there's always a space before open brackets and braces (( and {) and a space after (} and )), so that the style is consistent.

@edgardmessias
Copy link
Contributor Author

I'll Work on it already

@edgardmessias
Copy link
Contributor Author

Ready @abahgat

@abahgat
Copy link
Owner

abahgat commented Oct 2, 2014

Thanks, one more thing you might consider for the future, if you will, is to squash your cleanup and formatting changes with the original commits so that you have a cleaner history for your pull requests.

@edgardmessias
Copy link
Contributor Author

Thanks, I'll see how it works

@abahgat abahgat added the feature label Oct 2, 2014
@abahgat
Copy link
Owner

abahgat commented Oct 2, 2014

Change has been merged: 3b36845

Thank you again 👍

@abahgat abahgat closed this Oct 2, 2014
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.

2 participants