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

Allow selection of KeePassXC group when creating new entry #382

Closed
wohali opened this issue Jan 16, 2019 · 7 comments
Closed

Allow selection of KeePassXC group when creating new entry #382

wohali opened this issue Jan 16, 2019 · 7 comments

Comments

@wohali
Copy link

wohali commented Jan 16, 2019

Expected Behavior

When creating a new entry, I should be able to select the group within which I place the entry. For instance:

  • Work (Job 1)
  • Work (Job 2)
  • Personal
  • Financial
  • Government
  • Online Retailers
  • Gaming

etc.

These would map to the groups inside of KeePassXC.

The dialogue could even allow for creation of new groups inside of KeePassXC.

Current Behavior

All browser passwords are placed in a single group, KeePassXC-Browser Passwords. The user must then manually drag these to the group after creation - not convenient.

Possible Solution

Provide a popup window on creation of entry that allows selection of group.

Steps to Reproduce (for bugs)

Not a bug.

Debug info

Not a bug.

@varjolintu
Copy link
Member

This would be very nice. We need to discuss if this would be more convenient to add to the KeePassXC side instead.

@wohali
Copy link
Author

wohali commented Jan 22, 2019

I can see adding the bulk of the functionality to KeePassXC, but the point is that you can pick the group at the instant you create the new entry, which occurs in the dropdown in the browser.

Inspiration of course comes from Kee:

mwsnap 2019-01-22 10_38_16

@varjolintu
Copy link
Member

That would be possible too. KeePassXC 2.40 is released soon so I'll try to add this behaviour before that (just a basic support for a group parameter: if it's set, then use a group set by the user, otherwise use the default).

@droidmonkey
Copy link
Member

We can also aim for 2.4.1 to add the hook. But if you can make it ready before the beta this week that would be awesome.

@wohali
Copy link
Author

wohali commented Jan 22, 2019

@varjolintu would the group list be exposed to the browser plugin as well, to populate the dropdown? Icons are only a nice to have.

@varjolintu
Copy link
Member

@wohali That would be needed, yes.

@varjolintu
Copy link
Member

It's done. See #396 and keepassxreboot/keepassxc#2637.

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

3 participants