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

make carriage returns behave normally #7

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

Conversation

hughstephens
Copy link

OK so essentially we noticed a behaviour where carriage returns were behaving really abnormally, and it was common to see appium enter something like this

@someon
ehas a cat

rather than

@Someone
has a cat

The extra letter would drop between lines. It didn't happen too consistently etc, but we eventually worked out what it was, and changing the carriage return to be ascii printable seems to have fixed the error.

Was tested as still happily typing Emoji and other unicode characters so I don't think it will cause anyone else problems, but feel free to take it or leave it - wanted to add a PR here to give back, as we rely on Appium a lot and I'm more than happy to support it however we can!

@jlipps
Copy link
Contributor

jlipps commented Aug 21, 2015

thanks @hughstephens! let's have @imurchie take a look

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.

2 participants