diff --git a/asreview/webapp/src/Components/SignInForm.js b/asreview/webapp/src/Components/SignInForm.js index 51f8c7f1b..1b108157c 100644 --- a/asreview/webapp/src/Components/SignInForm.js +++ b/asreview/webapp/src/Components/SignInForm.js @@ -109,6 +109,7 @@ const SignInForm = (props) => { variant="outlined" fullWidth autoFocus + autoComplete="email" /> { variant="outlined" fullWidth type={returnType()} + autoComplete="current-password" /> { // be called when the form is submitted const navigate = useNavigate(); - const [showPassword, toggleShowPassword] = useToggle(); - - const returnType = () => { - return !showPassword ? "password" : "text"; - }; + const [showPassword, toggleShowPassword] = useToggle(false); const initialValues = { email: "", @@ -146,7 +143,6 @@ const SignUpForm = (props) => { spacing={3} component="form" noValidate - autoComplete="off" > { label="Email" size="small" type="email" + autoComplete="email" fullWidth value={formik.values.email} onChange={formik.handleChange} @@ -162,32 +159,6 @@ const SignUpForm = (props) => { {formik.touched.email && formik.errors.email ? ( {formik.errors.email} ) : null} - - {formik.touched.name && formik.errors.name ? ( - {formik.errors.name} - ) : null} - - {formik.touched.affiliation && formik.errors.affiliation ? ( - {formik.errors.affiliation} - ) : null} { size="small" autoComplete="new-password" fullWidth - type={returnType()} + type={showPassword ? "text" : "password"} value={formik.values.password} onChange={formik.handleChange} onBlur={formik.handleBlur} @@ -205,8 +176,9 @@ const SignUpForm = (props) => { id="confirmPassword" label="Confirm Password" size="small" + autoComplete="new-password" fullWidth - type={returnType()} + type={showPassword ? "text" : "password"} onKeyDown={handleEnterKey} value={formik.values.confirmPassword} onChange={formik.handleChange} @@ -254,7 +226,35 @@ const SignUpForm = (props) => { )} {isError && } - + + + {formik.touched.name && formik.errors.name ? ( + {formik.errors.name} + ) : null} + + {formik.touched.affiliation && formik.errors.affiliation ? ( + {formik.errors.affiliation} + ) : null}