Skip to content

Commit 00d2000

Browse files
committed
Updates to the Password Input
1 parent 5220205 commit 00d2000

File tree

1 file changed

+13
-16
lines changed

1 file changed

+13
-16
lines changed

lib/src/password-input/PasswordInput.tsx

Lines changed: 13 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -41,21 +41,16 @@ const DxcPasswordInput = React.forwardRef<RefType, PasswordInputPropsType>(
4141
const { passwordInput } = useTranslatedLabels();
4242

4343
useEffect(() => {
44-
setInputType("password", inputRef?.current);
45-
setAriaAttributes("false", passwordInput.inputShowPasswordTitle, inputRef?.current);
46-
}, [passwordInput]);
47-
48-
const viewPassword = () => {
49-
setIsPasswordVisible(true);
50-
setInputType("text", inputRef?.current);
51-
setAriaAttributes("true", passwordInput.inputHidePasswordTitle, inputRef?.current);
52-
};
53-
54-
const hidePassword = () => {
55-
setIsPasswordVisible(false);
56-
setInputType("password", inputRef?.current);
57-
setAriaAttributes("false", passwordInput.inputShowPasswordTitle, inputRef?.current);
58-
};
44+
(() => {
45+
if (isPasswordVisible) {
46+
setInputType("text", inputRef.current);
47+
setAriaAttributes("true", passwordInput.inputHidePasswordTitle, inputRef.current);
48+
} else {
49+
setInputType("password", inputRef.current);
50+
setAriaAttributes("false", passwordInput.inputShowPasswordTitle, inputRef.current);
51+
}
52+
})();
53+
}, [isPasswordVisible, passwordInput]);
5954

6055
return (
6156
<PasswordInput ref={ref}>
@@ -65,7 +60,9 @@ const DxcPasswordInput = React.forwardRef<RefType, PasswordInputPropsType>(
6560
value={value}
6661
helperText={helperText}
6762
action={{
68-
onClick: isPasswordVisible ? hidePassword : viewPassword,
63+
onClick: () => {
64+
setIsPasswordVisible((isPasswordVisible) => !isPasswordVisible);
65+
},
6966
icon: isPasswordVisible ? icons.hidePassword : icons.showPassword,
7067
title: isPasswordVisible ? passwordInput.inputHidePasswordTitle : passwordInput.inputShowPasswordTitle,
7168
}}

0 commit comments

Comments
 (0)