Skip to content

Add ⌘ + Enter keycommand #2

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

Merged
merged 1 commit into from
Jan 29, 2018
Merged

Add ⌘ + Enter keycommand #2

merged 1 commit into from
Jan 29, 2018

Conversation

BasThomas
Copy link
Collaborator

@BasThomas BasThomas commented Jan 8, 2018

Fixes #1.

Didn't want to "all out" like SlackTV atm; let me know what you think.

@rnystrom rnystrom self-requested a review January 16, 2018 01:35
@rnystrom
Copy link
Member

rnystrom commented Jan 20, 2018

@BasThomas GitHawk is just about pulling the remote repos. Quick question, does this disable using the return key on iOS? We still want to be able to enter newlines.

Also any interest setting up some unit tests for this?

@BasThomas
Copy link
Collaborator Author

BasThomas commented Jan 20, 2018

does this disable using the return key on iOS?

No; we need the ⌘ Modifier key for it to trigger.

Also any interest setting up some unit tests for this?

I'll take a look!

@rnystrom
Copy link
Member

Tests aren’t a big deal, lmk I can merge this now and revisit tests later!

Sent with GitHawk

@BasThomas
Copy link
Collaborator Author

Go ahead!

@rnystrom rnystrom merged commit e98f81c into GitHawkApp:master Jan 29, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

2 participants