diff --git a/src/app/pages/onboarding/sign-in/sign-in.tsx b/src/app/pages/onboarding/sign-in/sign-in.tsx index 6b430dae857..faad25e2dcb 100644 --- a/src/app/pages/onboarding/sign-in/sign-in.tsx +++ b/src/app/pages/onboarding/sign-in/sign-in.tsx @@ -1,4 +1,4 @@ -import { useState } from 'react'; +import { useEffect, useState } from 'react'; import { useNavigate } from 'react-router-dom'; import { RouteUrls } from '@shared/route-urls'; @@ -15,19 +15,23 @@ export function SignIn() { const navigate = useNavigate(); const [twentyFourWordMode, setTwentyFourWordMode] = useState(true); - const [mnemonic, setMnemonic] = useState<(string | null)[]>(() => createNullArrayOfLength(24)); + const [mnemonic, setMnemonic] = useState<(string | null)[]>([]); useRouteHeader(
navigate(RouteUrls.Onboarding)} hideActions />); + useEffect(() => { + const emptyMnemonicArray = twentyFourWordMode + ? createNullArrayOfLength(24) + : createNullArrayOfLength(12); + setMnemonic(emptyMnemonicArray); + }, [twentyFourWordMode]); + return ( <> { - setTwentyFourWordMode(!twentyFourWordMode); - setMnemonic(createNullArrayOfLength(twentyFourWordMode ? 24 : 12)); - }} + onClick={() => setTwentyFourWordMode(!twentyFourWordMode)} twentyFourWordMode={twentyFourWordMode} /> } diff --git a/tests/specs/onboarding/onboarding.spec.ts b/tests/specs/onboarding/onboarding.spec.ts index 93aabe19aa8..cfa55668eda 100644 --- a/tests/specs/onboarding/onboarding.spec.ts +++ b/tests/specs/onboarding/onboarding.spec.ts @@ -55,12 +55,10 @@ test.describe('Onboarding an existing user', () => { // enter some key partial const validPartialKey = 'shoulder any pencil'; await onboardingPage.signInMnemonicKey(validPartialKey); - const signInButton = await onboardingPage.page.getByTestId(OnboardingSelectors.SignInBtn); const signInSeedError = await onboardingPage.page.getByTestId( OnboardingSelectors.SignInSeedError ); await test.expect(signInSeedError).not.toBeVisible(); - await test.expect(signInButton).toBeDisabled(); }); test('Activity tab', async ({ extensionId, globalPage, onboardingPage, homePage }) => {