Add configurable tap action behaviour - #1786
Open
michaelschattgen wants to merge 3 commits into
Open
Conversation
michaelschattgen
marked this pull request as draft
April 14, 2026 21:16
michaelschattgen
force-pushed
the
feature/tap-config
branch
from
April 14, 2026 21:32
610b2ab to
99f613b
Compare
michaelschattgen
marked this pull request as ready for review
April 14, 2026 21:34
alexbakker
requested changes
Apr 24, 2026
alexbakker
left a comment
alexbakker
left a comment
Member
There was a problem hiding this comment.
I haven't taken a close look at the code yet, but I noticed some issues while doing some initial testing:
- Deselecting the final selected entry no longer closes the action menu
- Entries can no longer be unhighlighted/unrevealed manually
- Navigation to the tap actions preferences submenu is not animated
- Double tapping a hidden entry to copy it, reveals it.
- Settings: Tap to reveal = true, Highlight when tapped = false, Single tap = none, Double tap = Copy, Reserve first tap for reveal = false
- There appears to be a delay for single tap actions
- "Reserve first tap for reveal" also applies to highlighting an entry, but only if you first enable tap to reveal, enable "reserve first tap" and then disable tap to reveal again.
Comment on lines
173
to
464
| @@ -464,43 +478,9 @@ | |||
| entryHolder.itemView.setOnClickListener(new View.OnClickListener() { | |||
Member
There was a problem hiding this comment.
Why were these comments removed?
Comment on lines
914
to
952
Member
There was a problem hiding this comment.
Why were these comments removed?
Comment on lines
307
to
594
| @@ -396,7 +398,7 @@ | |||
| <string name="pref_groups_multiselect_summary">Allow the selection of multiple groups at the same time</string> | |||
| <string name="pref_minimize_on_copy_title">Minimize on copy</string> | |||
| <string name="pref_minimize_on_copy_summary">Minimize the app after copying a token</string> | |||
| <string name="pref_copy_behavior_title">Copy tokens to the clipboard</string> | |||
| <string name="pref_copy_behavior_title" tools:ignore="UnusedResources">Copy tokens to the clipboard</string> | |||
| <string name="pref_search_behavior_title">Search behavior</string> | |||
| <string name="pref_pause_entry_title">Freeze tokens when tapped</string> | |||
| <string name="pref_pause_entry_summary">Pause automatic refresh of tokens by tapping them. Tokens will not update as long as they are focused. Requires \"Highlight tokens when tapped\" or \"Tap to reveal\".</string> | |||
| @@ -539,10 +541,6 @@ | |||
| <string name="dialog_wipe_entries_message">Your vault already contains entries. Do you want to remove these entries before importing this file?\n\n<b>In doing so, you will permanently lose access to the existing entries in the vault.</b></string> | |||
| <string name="dialog_wipe_entries_checkbox">Wipe vault contents</string> | |||
|
|
|||
| <string name="import_from_clipboard_title">Import from clipboard</string> | |||
|
|
|||
| <string name="import_from_clipboard_message">We found a valid otpauth uri on your clipboard.\n\n<b>Name</b>: %1$s\n<b>Issuer</b>: %2$s\n\nDo you want to use it to prefill this entry?</string> | |||
|
|
|||
| <string name="panic_trigger_ignore_toast">Aegis received panic trigger but setting is disabled, ignoring</string> | |||
| <string name="pref_panic_trigger_title">Delete vault on panic trigger</string> | |||
| <string name="pref_panic_trigger_summary">Delete vault when a panic trigger is received from Ripple</string> | |||
| @@ -590,9 +588,9 @@ | |||
| <string name="pref_grouping_size_three">Groups of 3</string> | |||
| <string name="pref_grouping_size_four">Groups of 4</string> | |||
|
|
|||
| <string name="pref_copy_behavior_never">Never</string> | |||
| <string name="pref_copy_behavior_single_tap">Single tap</string> | |||
| <string name="pref_copy_behavior_double_tap">Double tap</string> | |||
| <string name="pref_copy_behavior_never" tools:ignore="UnusedResources">Never</string> | |||
| <string name="pref_copy_behavior_single_tap" tools:ignore="UnusedResources">Single tap</string> | |||
| <string name="pref_copy_behavior_double_tap" tools:ignore="UnusedResources">Double tap</string> | |||
|
|
|||
Member
There was a problem hiding this comment.
Let's remove strings that are no longer used.
Comment on lines
+25
to
+30
| <Preference | ||
| android:key="pref_tap_actions_hint" | ||
| android:title="@string/pref_tap_actions_hint_title" | ||
| android:summary="@string/pref_tap_actions_hint_summary" | ||
| android:selectable="false" | ||
| app:iconSpaceReserved="false" /> |
Member
There was a problem hiding this comment.
I think this looks a little bit too much like an actual preference you can tap. I'd suggest making this look the same as the D2D hint we have at the bottom of the Backups preference screen.
Comment on lines
914
to
952
| public int getItemCount() { | ||
| // Always at least one item because of the footer | ||
| // Two in case there's also an error card | ||
| int baseCount = 1; | ||
| if (isErrorCardShown()) { | ||
| baseCount++; | ||
| } | ||
|
|
||
| return baseCount + getShownEntries().size(); | ||
| } | ||
|
|
||
| @Nullable | ||
| public ErrorCardInfo getErrorCardInfo() { | ||
| return _errorCardInfo; | ||
| } | ||
|
|
||
| public boolean isErrorCardShown() { | ||
| return _errorCardInfo != null; | ||
| } | ||
|
|
||
| public boolean isPositionErrorCard(int position) { | ||
| return isErrorCardShown() && position == 0; | ||
| } | ||
|
|
||
| public boolean isPositionFooter(int position) { | ||
| return position == (getItemCount() - 1); | ||
| } | ||
|
|
||
| /** | ||
| * Translates the given entry position in the recycler view, to its index in the shown entries list. | ||
| */ | ||
| public int translateEntryPosToIndex(int position) { | ||
| if (position == NO_POSITION) { | ||
| return NO_POSITION; | ||
| } | ||
|
|
||
| if (isErrorCardShown()) { | ||
| position -= 1; | ||
| } | ||
|
|
||
| return position; | ||
| } | ||
|
|
||
| /** | ||
| * Translates the given entry index in the shown entries list, to its position in the recycler view. | ||
| */ | ||
| public int translateEntryIndexToPos(int index) { |
Member
There was a problem hiding this comment.
Why were these comments removed?
| <?xml version="1.0" encoding="utf-8"?> | ||
| <PreferenceScreen xmlns:android="http://schemas.android.com/apk/res/android" | ||
| xmlns:app="http://schemas.android.com/apk/res-auto" | ||
| android:title="@string/pref_tap_actions_title"> |
Member
There was a problem hiding this comment.
This title does not appear to get applied.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR adds the ability to set the single tap and double tap behaviour respectively. This allows us to branch out to other tap actions like the one mentioned in (but not exclusive to) #1688.
Hopefully this PR fixes issues like the ones described in #1223, #1569, #1064, #1667 and probably many more that I can't find right now.
I also added a preferences migration for people who already have set up their 'Copy to clipboard' to either single tap or double tap.