Fixed a few instances of useEffect misuse

The first parameters were using an arrow function expression without
braces, so the internal expression's return value was being used, and
wasn't a cleanup function for useEffect. This commit should cover all
misuses.

ui/components/Password/PasswordStrengthBar.tsx

@@ -16,7 +16,9 @@ export default function PasswordStrengthBar(
   const { t } = useTranslation()
   const [evaluation, setEvaluation] = useState(0)
 
-  useEffect(() => setEvaluation(zxcvbn(password).score), [password])
+  useEffect(() => {
+    setEvaluation(zxcvbn(password).score)
+  }, [password])
 
   const getDescription = useCallback(() => {
     if (!password) return t("passwordStrength.strength")

ui/components/Shared/SharedAddressInput.tsx

@@ -37,10 +37,9 @@ export default function SharedAddressInput({
   const { errorMessage, handleInputChange, isValidating } =
     useAddressOrNameValidation(onAddressChange)
 
-  useEffect(
-    () => handleInputChange(debouncedValue),
-    [debouncedValue, handleInputChange],
-  )
+  useEffect(() => {
+    handleInputChange(debouncedValue)
+  }, [debouncedValue, handleInputChange])
 
   return (
     <>

ui/components/Signing/Signer/TransactionButton.tsx

@@ -57,10 +57,9 @@ export default function TransactionButton({
     }
   }
 
-  useEffect(
-    () => setUnlockButtons(isTransactionDataReady),
-    [isTransactionDataReady, setUnlockButtons],
-  )
+  useEffect(() => {
+    setUnlockButtons(isTransactionDataReady)
+  }, [isTransactionDataReady, setUnlockButtons])
 
   useEffect(() => {
     const increaseFocusChangeNonce = () => {