feat(flutterfire_ui): add autofillhints (#7668)

* feat(flutterfire_ui): add autofillhints * feat: change prop of PasswordInput to autofill with default value
firebase · Jan 31, 2022 · dd94e93 · dd94e93
1 parent 0c43bfc
commit dd94e93
Show file tree

Hide file tree

Showing 6 changed files with 22 additions and 6 deletions.
diff --git a/packages/flutterfire_ui/lib/src/auth/widgets/email_form.dart b/packages/flutterfire_ui/lib/src/auth/widgets/email_form.dart
@@ -160,6 +160,7 @@ class _SignInFormContentState extends State<_SignInFormContent> {
           widget.action == AuthAction.link) ...[
         const SizedBox(height: 8),
         PasswordInput(
+          autofillHints: const [AutofillHints.newPassword],
           focusNode: confirmPasswordFocusNode,
           controller: confirmPasswordCtrl,
           onSubmit: _submit,
@@ -204,12 +205,14 @@ class _SignInFormContentState extends State<_SignInFormContent> {
       ),
     ];
 
-    return Form(
-      key: formKey,
-      child: Column(
-        crossAxisAlignment: CrossAxisAlignment.stretch,
-        mainAxisSize: MainAxisSize.min,
-        children: children,
+    return AutofillGroup(
+      child: Form(
+        key: formKey,
+        child: Column(
+          crossAxisAlignment: CrossAxisAlignment.stretch,
+          mainAxisSize: MainAxisSize.min,
+          children: children,
+        ),
       ),
     );
   }

diff --git a/packages/flutterfire_ui/lib/src/auth/widgets/email_input.dart b/packages/flutterfire_ui/lib/src/auth/widgets/email_input.dart
@@ -26,6 +26,7 @@ class EmailInput extends StatelessWidget {
     final l = FlutterFireUILocalizations.labelsOf(context);
 
     return UniversalTextFormField(
+      autofillHints: const [AutofillHints.email],
       autofocus: autofocus ?? false,
       focusNode: focusNode,
       controller: controller,

diff --git a/packages/flutterfire_ui/lib/src/auth/widgets/internal/universal_text_form_field.dart b/packages/flutterfire_ui/lib/src/auth/widgets/internal/universal_text_form_field.dart
@@ -16,6 +16,7 @@ class UniversalTextFormField extends PlatformWidget {
   final bool? enableSuggestions;
   final bool? autocorrect;
   final Widget? prefix;
+  final Iterable<String>? autofillHints;
 
   const UniversalTextFormField({
     Key? key,
@@ -31,6 +32,7 @@ class UniversalTextFormField extends PlatformWidget {
     this.focusNode,
     this.enableSuggestions,
     this.autocorrect,
+    this.autofillHints,
   }) : super(key: key);
 
   @override
@@ -45,6 +47,7 @@ class UniversalTextFormField extends PlatformWidget {
         ),
       ),
       child: CupertinoTextFormFieldRow(
+        autofillHints: autofillHints,
         focusNode: focusNode,
         padding: EdgeInsets.zero,
         controller: controller,
@@ -63,6 +66,7 @@ class UniversalTextFormField extends PlatformWidget {
   @override
   Widget buildMaterial(BuildContext context) {
     return TextFormField(
+      autofillHints: autofillHints,
       autofocus: autofocus ?? false,
       focusNode: focusNode,
       controller: controller,

diff --git a/packages/flutterfire_ui/lib/src/auth/widgets/password_input.dart b/packages/flutterfire_ui/lib/src/auth/widgets/password_input.dart
@@ -10,13 +10,15 @@ class PasswordInput extends StatelessWidget {
   final void Function(String value) onSubmit;
   final String label;
   final String? Function(String? value)? validator;
+  final Iterable<String> autofillHints;
 
   const PasswordInput({
     Key? key,
     required this.focusNode,
     required this.controller,
     required this.onSubmit,
     required this.label,
+    this.autofillHints = const [AutofillHints.password],
     this.validator,
   }) : super(key: key);
 
@@ -25,6 +27,7 @@ class PasswordInput extends StatelessWidget {
     final l = FlutterFireUILocalizations.labelsOf(context);
 
     return UniversalTextFormField(
+      autofillHints: autofillHints,
       focusNode: focusNode,
       controller: controller,
       obscureText: true,

diff --git a/packages/flutterfire_ui/lib/src/auth/widgets/phone_input.dart b/packages/flutterfire_ui/lib/src/auth/widgets/phone_input.dart
@@ -265,6 +265,9 @@ class PhoneInputState extends State<PhoneInput> {
               SizedBox(
                 width: 90,
                 child: UniversalTextFormField(
+                  autofillHints: const [
+                    AutofillHints.telephoneNumberCountryCode
+                  ],
                   autofocus: false,
                   controller: countryController,
                   prefix: const Text('+'),
@@ -282,6 +285,7 @@ class PhoneInputState extends State<PhoneInput> {
               const SizedBox(width: 8),
               Expanded(
                 child: UniversalTextFormField(
+                  autofillHints: const [AutofillHints.telephoneNumberNational],
                   autofocus: true,
                   focusNode: numberFocusNode,
                   controller: numberController,

diff --git a/packages/flutterfire_ui/lib/src/auth/widgets/sms_code_input.dart b/packages/flutterfire_ui/lib/src/auth/widgets/sms_code_input.dart
@@ -220,6 +220,7 @@ class SMSCodeInputState extends State<SMSCodeInput> {
             child: Padding(
               padding: const EdgeInsets.only(top: 30),
               child: UniversalTextFormField(
+                autofillHints: const [AutofillHints.oneTimeCode],
                 autofocus: true,
                 focusNode: focusNode,
                 controller: controller,