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

Autotype now includes window title per default #1814

Open
dploeger opened this issue May 12, 2021 · 20 comments
Open

Autotype now includes window title per default #1814

dploeger opened this issue May 12, 2021 · 20 comments
Assignees
Labels
› Auto-type Auto-type functionality in desktop apps Build ◦ Desktop Specific to desktop Type ◦ Bug Something isn't working
Milestone

Comments

@dploeger
Copy link

Describe the bug
I just upgraded from 1.17.6 to 1.18.3 and noticed, that Autotype includes filtering for the window title now and that breaks my workflow, because I have "Google" as a password in my database and now every password I want to autotype in Chrome only shows that google password (if I haven't added the specific website to the correct item).

I'm pretty sure, window titles were filtered before as well, but somehow this problem didn't show. I'm also unsure on how to fix this as the title can just as well be a good filter option. So in the solution I vouch for a configuration option which default filters to include for autotype. (Does that make it a FR instead?)

To Reproduce
Steps to reproduce the behavior:

  1. Add a new password that includes "Google" in its name.
  2. Open a website in Google Chrome
  3. Hit autotype
  4. Only see the Google password

Expected behavior
It could be configurable wether the window title is used as a default filter option for Autotype.

Environment
KeeWeb v1.18.3 (337ef44, 2021-05-09)
Environment: electron v12.0.7
User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 11_3_1) AppleWebKit/537.36 (KHTML, like Gecko) KeeWeb/1.18.3 Chrome/89.0.4389.128 Electron/12.0.7 Safari/537.36

@dploeger dploeger added the Type ◦ Bug Something isn't working label May 12, 2021
@antelle antelle added › Auto-type Auto-type functionality in desktop apps Build ◦ Desktop Specific to desktop labels May 12, 2021
@antelle
Copy link
Member

antelle commented May 12, 2021

Interesting, I wonder how to fix this better. Probably we need a setting because I can imagine why someone would like to have the filter enabled by default.

It's also possible to remove standard browser names, such as Google Chrome, Firefox, etc... This can be another improvement.

@antelle antelle added this to the v1.18.x milestone May 12, 2021
@jivarson
Copy link

jivarson commented May 12, 2021

@dploeger, what's your stand on the below suggestions (coming from #1815)?

A. Hotkey in autotype-selection screen (Ctrl+A or such) to temporary disable all filters
B. Persistent setting to enable / disable all filters
C. Persistent setting to show / hide filters (by default).

from my POV, the order is by preference, but I mainly want to get rid of the mouseclicks

@antelle
Copy link
Member

antelle commented May 12, 2021

There will be no preference to show / hide filters. However, we can add a preference to show the Title filter as disabled by default because it seems to be what causes troubles.

@dploeger
Copy link
Author

@jivarson I'm not sure if I really get those options. My opinion would be a preference to enable/disable certain filter types. That way I could just disable the window title filter for my specific problem.

Your idea about removing common browser names from the window title, @antelle, is a bit "hacky" in my opinion. That would be very hard to manage I think.

But I really have no really optimal solution here tbh.

@antelle
Copy link
Member

antelle commented May 12, 2021

I agree, it's possible that it's "hacky", I don't like this idea either, so probably it's better not to go there.

Looks like for now an option to disable the title filter would solve the problem.

@jivarson
Copy link

@jivarson I'm not sure if I really get those options. My opinion would be a preference to enable/disable certain filter types. That way I could just disable the window title filter for my specific problem.

Your idea about removing common browser names from the window title, @antelle, is a bit "hacky" in my opinion. That would be very hard to manage I think.

But I really have no really optimal solution here tbh.

Realized I mixed the terms disable\hide above, sorry.
I think we're agreeing that a persistent setting to 'untick' the filter by default (maybe only the Title-filter as @antelle said) is the way to go..

@antelle
Copy link
Member

antelle commented May 12, 2021

Added the setting, let's see if it makes it easier, I'll build a release with it this week.

@antelle antelle self-assigned this May 12, 2021
@antelle
Copy link
Member

antelle commented May 12, 2021

It's published: https://github.com/keeweb/keeweb/releases/latest

@dploeger
Copy link
Author

Works like a charm. Thanks a lot!

@jivarson
Copy link

Perfectamento!

@kaibrune
Copy link

kaibrune commented Sep 6, 2021

@antelle Sorry to ask but where can we find this option? By me it's still enabled by default and can't find a disable filter option. Thanks!

@antelle
Copy link
Member

antelle commented Sep 6, 2021

@kaibrune it's in Settings / General / Function. The filters won't disappear, but they won't be selected automatically.

@kaibrune
Copy link

kaibrune commented Sep 7, 2021

@kaibrune it's in Settings / General / Function. The filters won't disappear, but they won't be selected automatically.

Screenshot

Am I missing something? The "Filter entries by title in auto-type" is unchecked and still is the filter applied in the auto-type menu?

@antelle
Copy link
Member

antelle commented Sep 7, 2021

Which filter do you see?

@kaibrune
Copy link

kaibrune commented Sep 7, 2021

Which filter do you see?

image

I guess the window title filter? Which is activated by default, eventhough the setting is turned off.

@antelle
Copy link
Member

antelle commented Sep 7, 2021

@kaibrune I see, it happens because there's no URL. Then the setting is not applied, it should be fixed of course.

@antelle antelle reopened this Sep 7, 2021
@kaibrune
Copy link

kaibrune commented Sep 7, 2021

@kaibrune I see, it happens because there's no URL. Then the setting is not applied, it should be fixed of course.

Ahh that's a point. For logins without a URL the filter is still applied. Would be awesome to disable it globally. :)

@jivarson
Copy link

@kaibrune I see, it happens because there's no URL. Then the setting is not applied, it should be fixed of course.

This has been on my mind too, but I haven't taken the time to connect the dots. A fix would definitely speed up auto type on desktop apps.

@kaibrune
Copy link

Any updates on this matter? This is really blocking desktop use.

@XIDAGmbH
Copy link

XIDAGmbH commented Jun 6, 2023

I think this has still not been changed/fixed?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
› Auto-type Auto-type functionality in desktop apps Build ◦ Desktop Specific to desktop Type ◦ Bug Something isn't working
Projects
None yet
Development

No branches or pull requests

5 participants