8000 Autofill ime option for password managers. by kv1dr · Pull Request #9007 · thunderbird/thunderbird-android · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Autofill ime option for password managers. #9007

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

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

kv1dr
Copy link
@kv1dr kv1dr commented Mar 28, 2025

This fixes #7404: Add new account - autofill ime option for password managers. This pull request contains non stable version of androidx.autofill:autofill and androidx.compose.ui:ui package, so it shouldn't be merged yet. As @kewisch mentioned in issue, this pull request should be marked as blocked and it will be merged, when those 2 packages become stable. Will keep an eye on new releases.

Tested with keepass. Attaching screenshots of results:

Screenshot_20250328_222319_Thunderbird Debug
Screenshot_20250328_222350_Thunderbird Debug
Screenshot_20250328_222422_Thunderbird Debug
Screenshot_20250328_222500_Thunderbird Debug
Screenshot_20250328_222518_Thunderbird Debug

@kv1dr kv1dr requested review from kewisch and wmontwe as code owners March 28, 2025 22:05
@kv1dr kv1dr marked this pull request as draft March 28, 2025 22:07
@kewisch
Copy link
Member
kewisch commented Mar 31, 2025

Blocked as per discussion in #7404

We need androidx.autofill:autofill version 1.3.0 to be released which will also require us to use androidx.compose.ui version 1.8.0, which has also not yet been released.

@kewisch
Copy link
Member
kewisch commented May 5, 2025

@wmontwe FYI this issue when you do your dependency update run

@wmontwe
Copy link
Member
wmontwe commented May 6, 2025

We're now using Jetpack Compose 1.8 and could continue with this pull-request.

kv1dr added 2 commits May 6, 2025 16:52
# Conflicts:
#	gradle/libs.versions.toml
@kv1dr
Copy link
Author
kv1dr commented May 6, 2025

Merged main branch into my branch. I will mark this pull request as ready for review. Please note that androidx.autofill:autofill is still 1.3.0-rc1 so feel free to wait for stable release.

@kv1dr kv1dr marked this pull request as ready for review May 6, 2025 15:15
@kewisch kewisch assigned asoucar and unassigned wmontwe May 6, 2025
@kewisch
Copy link
Member
kewisch commented May 6, 2025

Personally I think we could risk a rc here, but I'd prefer someone decide that has more experience with Android rc quality.

@kewisch kewisch assigned rafaeltonholo and unassigned asoucar May 9, 2025
@rafaeltonholo
Copy link
Member

The only issue I encountered while testing this PR is that the email is not autofilled when using 1Password, although it works fine with Google Password.

Additionally, if we decide to implement this in the application, we will need to make a change to the IncomingFormItems.kt file, specifically under the username TextInput, by adding ContentType.Username + ContentType.Email.

My only concern about introducing an RC version is the risk of the application crashing if an uncaught bug exists before the stable release. However, since the autofill feature is only utilized on two screens during the configuration process and does not affect the input behaviour if autofill is not used, I am comfortable with proceeding.

It might also be worth hearing @wmontwe's opinion on this.

Let me know your thoughts on this.

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.

Add new account - autofill ime option for password managers
5 participants
0