Copy user and leave window open after action

[zoidy]

I have the following use case: It is sometimes useful to copy both the username and password instead of autotyping (e.g., if the password field doesn't show up until after the username is entered, or if one uses a remapped keyboard which causes autotype to enter the wrong characters)

I have added these functions to AutoTypeSearch
* The ability to copy the user name as a default or alternative action
* The option to leave the search window open after executing the primary action. This enables setting copy user as the primary action and copy password as the secondary without having to re-do the search.

I have tested it for my uses and it works. The plugin is attached and the source is here https://github.com/zoidy/autotypesearch. Feel free to add the changes to the main release

[AlexVallat]

Thanks, that looks good, but I think it might be a bit confusing that the primary action doesn't close and the secondary one does. Also, it won't automatically close on loss of focus either (logical, as you wouldn't be able to paste the username in without having it close otherwise).

Perhaps it would be better to have the option as just "Leave window open", and the user has to click escape to close it once they've finished? That way it doesn't matter if they use the primary or secondary action first, or how many times, and it won't be surprising that the window stays open when deactivated.

I do not think it is possible to do a pull request from GitHub into Sourceforge, but if you wanted to fork it on Sourceforge then it would work. This would be best if you intend to do further work to be merged. If not, let me know and I can just copy your changes manually from your GitHub fork.

[zoidy]

Hi, I've opened a merge request on Sourceforge. I've added the ability for the user to separately control whether the search window closes after executing the primary or alternative action. The options are off by default which replicates the existing behavior. Hopefully this will avoid confusion.

Additionally, the window focus should behave more like the current version of the plugin. When one of the new options are enabled, the window will close when it loses focus unless the corresponding primary or alternative action is executed.

[AlexVallat]

Thank you for the PR. I decided to go a different way with the fix for autotype, there were just too many edge cases with the ActiveForm solution in your PR, so I just went the simple way of hiding and then re-showing the SearchWindow, if it's visible.

I still prefer to have a single option that keeps the window open until manually closed, I think that covers all use cases of why people might want the window left open without trying to configure each one individually.