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

Add text to avoid empty input for input workaround #888

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

Conversation

soraxas
Copy link

@soraxas soraxas commented Feb 8, 2024

For the input workaround, we add a space key event to input some text before pressing the enter key. This avoid the error that happens due to gtk preventing empty input.

This make it so that the workaround works in gtk auth like:

Screenshot from 2024-02-06 17-18-11

Pretty much after the original workaround doesn't work, we will enter a space key before pressing enter

For the `input` workaround, we add a space key event to input some text
before pressing the enter key. This avoid the error that happens due to
gtk preventing empty input.

Signed-off-by: Tin Lai <oscar@tinyiu.com>
@soraxas
Copy link
Author

soraxas commented Feb 8, 2024

closes #735

@musikid musikid self-requested a review April 9, 2024 10:09
Comment on lines +412 to +415
// try it one more time with an non-empty input field
enter_device.send_space_press();
enter_device.send_enter_press();

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This part should be either in the first send try or in the loop.

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.

None yet

2 participants