Skip to content
This repository has been archived by the owner on Dec 9, 2021. It is now read-only.

implement {DELAY=X} in autotype #161

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

implement {DELAY=X} in autotype #161

wants to merge 1 commit into from

Conversation

jmper
Copy link

@jmper jmper commented May 6, 2016

{DELAY=X} sequence inserts Xms delay between each character during
autotype

{DELAY=X} sequence inserts Xms delay between each character during
autotype
@nmschulte
Copy link

nmschulte commented Jun 1, 2016

KeePassX already supports this feature, with a nearly identical syntax:

{DELAY X} -- no =, and X is in milliseconds.

Ah, I see that your syntax (with =) is a different feature:
https://www.keepassx.org/dev/issues/296

@jmper
Copy link
Author

jmper commented Jun 1, 2016

{DELAY X} and {DELAY=X} are different. {DELAY X} only inserts delay before autotyping the entire string. {DELAY=X} (which I added) enables entering delays between every character that is being autotyped. This is useful on sites which do some JS processing of entered characters and KeePassX is too fast for them.
Both DELAY features were meant to be compatible with original KeePass (and KeePassX 0.x), but so far only the first one was implemented.

@arigit
Copy link

arigit commented Jun 1, 2016

this is awesome news jmper, thank you for this. This was needed for true compatibility with Keepass, I have one particular site where this is needed and I found the missing support the hard way.

@arigit
Copy link

arigit commented Jun 1, 2016

p.s. I'm surpised there are 50+ PRs pending review by the devs dating back 9 months ago.
Hoping that this one can be merged soon :|

@nmschulte
Copy link

p.s. I'm surpised there are 50+ PRs pending review by the devs dating back 9 months ago.
Hoping that this one can be merged soon :|

There was a recent commit to the repo, so hopefully development will pick up again. If not, I'll probably implement a fix for global auto-type prompting to unlock the DB, or revert back to < v2.0.

@george-viaud
Copy link

This is much needed - I'm running into problems (likely due to plugins in my browser) that are causing the browser to "miss" autotyped characters ~50% of the time which is unfun. I suspect additional delay would fix this

Copy link

@george-viaud george-viaud left a comment

Choose a reason for hiding this comment

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

Working great - thanks!

Suggest a sequence similar to:

{DELAY=15}{DELAY 200}{USERNAME}{DELAY 100}{TAB}{DELAY 100}{PASSWORD}

@george-viaud
Copy link

george-viaud commented Nov 3, 2016

@jmper would you consider please making this pull request also on fork:

https://github.com/keepassxreboot/keepassxc

It appears they've either missed it or perhaps haven't put it in their main branch yet (not sure how to check)

Many thanks


EDIT

Thanks @jmper - looks like your PR got implemented there - very good indeed!

daniellandau pushed a commit to daniellandau/keepassx that referenced this pull request Feb 9, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants