PhilippC / PhilippC/keepass2android
[FEAT] Do not show entry selection dialog
Nobody has claimed this yet.
- Dominant language
- C#
- Stars
- 6.2k
- Forks
- 478
- Avg merge
- 1h 4m
- Merged PRs (30d)
- 2
Description
The entry selection dialog shows up after tapping the lock in the KP2A keyboard and has the following options:
- Select entry
- Search for entry with the app's package name
It is quite irritating that I have always have to choose "Select entry" coming from the web browser. It unnecessarily increases the number of actions and the time required to enter credentials. I can think of the following improvements:
- Replace the dialog with "Search for entry with the app's package name" option at the top of the credential list shown after unlocking database. This way, if the user wants to select an entry, they can do it right away, and if they want to search, they can also do it in the same number of actions as previously.
- Add an option to make either choice the default and thus never show the dialog. This is worse but probably simplest to implement. It would be good enough for me, though.
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by reproducing the entry selection dialog after tapping the lock in the KP2A keyboard, then trace the credential-list flow after unlocking the database. Compare the proposed default-selection and credential-list approaches, and confirm that selecting or searching for an entry takes no unnecessary extra action.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- android, csharp
- Domain
- mobile
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100