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 gboard issue #1087

Merged
merged 1 commit into from
Oct 31, 2017
Merged

fix gboard issue #1087

merged 1 commit into from
Oct 31, 2017

Conversation

115ek
Copy link
Contributor

@115ek 115ek commented Oct 31, 2017

conversion is now triggered by space after smiley

This fixes the google keyboard problem: every ASCII smiley needs a following space to confirm the conversion.
Now "ASCII_smiley" + "space" is converted into "emoji" + "space".
Is that okay or should we do it that way: "ASCII_smiley" + "space" is converted into "emoji" ?

conversion is now triggered by space after smiley
@daniele-athome daniele-athome self-requested a review October 31, 2017 20:59
@daniele-athome daniele-athome self-assigned this Oct 31, 2017
@daniele-athome daniele-athome added the enhancement New feature or request label Oct 31, 2017
@daniele-athome daniele-athome added this to the 4.1.0 milestone Oct 31, 2017
@daniele-athome
Copy link
Member

Thanks @115ek, this will do just fine!

@daniele-athome daniele-athome merged commit 8f06835 into kontalk:master Oct 31, 2017
@webratte
Copy link
Contributor

webratte commented Nov 1, 2017 via email

@115ek
Copy link
Contributor Author

115ek commented Nov 1, 2017

I'm not an expert but it's really great to contribute even small things to Kontalk.
@webratte And thanks for your replay - thats really motivating 😄
Although I didn't really contributed something compared to daniele and several other people ;)

@daniele-athome
Copy link
Member

If you'd like to continue on this road, there are many things to be done. I've recently begun to mark issues with labels help wanted and good first issue. The latter are easier to get on so you can get more into the architecture of the Kontalk app.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants