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

Support Deselecting Tokens #244

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

merktassel
Copy link
Contributor

@merktassel merktassel commented Jul 17, 2021

Currently it’s not possible to deselect a token once it’s been selected. This means that once you select a token, you must wait for it to dismiss itself when the timer is up, or force quit the application and re-open it to hide the token or use the share button.

This pull request adds the ability to tap on an empty area of the collection view to deselect tokens. I tested to ensure that no other behavior was changed, and existing gestures (tapping to view a token, swiping to delete, and dragging and dropping to rearrange) still work. It also animates as expected and only activates if the empty space of the collection view is tapped, not when the navigation bar or other elements are tapped so as to be intuitive for users.

Support.Deselecting.Tokens.mp4

@justin-stephenson
Copy link
Collaborator

Hi, thank you for the PR. This looks like a nice feature, but my concern is the user experience when the tokens list is full of tokens. I don't see much, if any empty space available in this case.

@merktassel
Copy link
Contributor Author

Hi, thank you for the PR. This looks like a nice feature, but my concern is the user experience when the tokens list is full of tokens. I don't see much, if any empty space available in this case.

Good point. I'll do some testing with more tokens configured and see what I can come up with. I'm open to any suggestions you may have.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants