Skip to content

Conversation

@SUPERCILEX
Copy link
Collaborator

@samtstern I've got a string of PRs lined up. This one updates deps, adds explicit autofill support, and does Android O cleanup.

Signed-off-by: Alex Saveau <saveau.alexandre@gmail.com>
Signed-off-by: Alex Saveau <saveau.alexandre@gmail.com>
Signed-off-by: Alex Saveau <saveau.alexandre@gmail.com>
tools:ignore="ResourceName">

<style name="FirebaseUI.TextInputEditText.EmailField">
<item name="android:autofillHints">emailAddress</item>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hmm I wonder how this will play with smartlock. It could be annoying if they just turned down the smartlock hint and then autofill offers them a new hint.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh, thanks for reminding me! I was going to disable password autofill if smart lock is enabled (because getting a save password prompt twice is really annoying).

As for the email, we could also disable autofill anytime smart lock is enabled because the only way the user could have reached the email field is if they didn't take the hint with smart lock enabled.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think that's a good idea, when smartlock is enabled disable other autofills. That's the path of least annoyance.

Signed-off-by: Alex Saveau <saveau.alexandre@gmail.com>
@samtstern
Copy link
Contributor

Woot!

@samtstern samtstern merged commit 2eb8050 into firebase:version-2.2.1-dev Aug 3, 2017
@SUPERCILEX SUPERCILEX deleted the updates branch August 3, 2017 22:25
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