Update Password Validation

[cbhernan]

What does this PR do?

This PR consolidates the Password field validations to use a single function to validate input. It also adds the icons to the password fields so that the user can toggle between password and text view on their input. It also enables the Change Password button to be enabled once all fields are filled.

Issue

#3

[Paz-X]

Hey @cbhernan , can you try the following code?

bool passwordVisible=false;  

TextField(
                  obscureText: passwordVisible,
                  decoration: InputDecoration(
                    border: UnderlineInputBorder(),
                    hintText: "Password",
                    labelText: "Password",
                    helperText:"Password must contain special character",
                    helperStyle:TextStyle(color:Colors.green),
                    suffixIcon: IconButton(
                      icon: Icon(passwordVisible
                          ? Icons.visibility
                          : Icons.visibility_off),
                      onPressed: () {
                        setState(
                          () {
                            passwordVisible = !passwordVisible;
                          },
                        );
                      },
                    ),
                    alignLabelWithHint: false,
                    filled: true,
                  ),
                  keyboardType: TextInputType.visiblePassword,
                  textInputAction: TextInputAction.done,
                ),
              ), 
      ),

I noticed you are using variables to determine if the password is visible or not. I think it can all be added to the widget itself instead of the class.

https://www.geeksforgeeks.org/flutter-show-hide-password-in-textfield/

I agree with the validator, we should try to make it into a function.

Also, let's remove import 'package:flutter/cupertino.dart'; since that is iOS specific.

[cbhernan]

@Paz-X Thank you for finding the correct icons, I've replaced them and removed the Cupertino icons.

You're right that I could just use the variable instead of using a ternary for obscureText so I'll make that update.

The validator is currently a function, it's called validatePasswords and can be found here

Regarding your suggestion, do you want me to replace what's currently on the branch with it?

[nickchaula]

LGTM