Skip to content
This repository has been archived by the owner on Apr 10, 2019. It is now read-only.

Password confirm behavior #5

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

Fishrock123
Copy link

Makes some adjustments to the password confirm flow.

  • Updates the confirm test to be a bit more clear.
  • Adds a "Cancel" button when in the confirm flow - it is unclear the emoji can be interacted with.

@indutny :)

This should make it a bit more clear that if you are at this point, you
are making a _new_ password.
They were offset horizontally a bit.
This functionality _is_ available by tapping the emojis, but that isn't
clear. No visual signifier or UI affordance is given to the emoji.

Adding a separate, iOS-normal button should alleviate this.

Fun fact: I was unaware the emoji could be tapped until I dug through
the source. :P
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant