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

Use of Ctrl+C to copy text can cause accidental deregister of the client #107

Open
maclobdell opened this issue Feb 2, 2020 · 4 comments

Comments

@maclobdell
Copy link
Contributor

Background:
For Windows users, it is common to use Ctrl+C to copy text. If someone uses this command to try to copy text from the serial terminal window (example: coping the device ID), then it causes the client to deregister. This can be confusing.

Suggestion:
Use a less command command, such as Ctrl+X to trigger the deregister.

I have tested this, and it works.

        } else if (in_char > 0 && in_char != 0x18) { // Ctrl+X is 0x18
            button_press(); // Simulate button press
            continue;
        }
        // if Ctrl+X, then reach this point and trigger deregister
        deregister_client();
        break;
@ciarmcom
Copy link
Member

ciarmcom commented Feb 2, 2020

Internal Jira reference: https://jira.arm.com/browse/IOTPART-9036

@MarceloSalazar
Copy link

@maclobdell thanks for sharing this - I agree with the use case and proposal to change to Ctrl+X

@JanneKiiskila
Copy link

Yes, have to agree. It's a quite common use-case to use the copy-paste, so this makes perfectly sense.

@JanneKiiskila
Copy link

@maclobdell - if you've already tested it, you could do a PR and let's get it merged in. Seems you have the code changes in your work area is, that way credit goes where it's due. :-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants