From f584f56d9c0a61f603445d4b3b470e6a30723654 Mon Sep 17 00:00:00 2001 From: David Walsh Date: Wed, 26 Jul 2023 20:03:30 -0500 Subject: [PATCH] UX: Allow quick Add Account name based on default name (#20168) * UX: Allow quick Add Account name * do not allow account names with just spaces --------- Co-authored-by: Howard Braham --- ui/components/multichain/create-account/create-account.js | 5 +++-- ui/components/ui/editable-label/editable-label.js | 2 +- 2 files changed, 4 insertions(+), 3 deletions(-) diff --git a/ui/components/multichain/create-account/create-account.js b/ui/components/multichain/create-account/create-account.js index ccd181164..838609ab7 100644 --- a/ui/components/multichain/create-account/create-account.js +++ b/ui/components/multichain/create-account/create-account.js @@ -39,11 +39,12 @@ export const CreateAccount = ({ onActionComplete }) => { const defaultAccountName = t('newAccountNumberName', [newAccountNumber]); const [newAccountName, setNewAccountName] = useState(''); + const trimmedAccountName = newAccountName.trim(); const { isValidAccountName, errorMessage } = getAccountNameErrorMessage( accounts, { t }, - newAccountName, + trimmedAccountName ?? defaultAccountName, defaultAccountName, ); @@ -58,7 +59,7 @@ export const CreateAccount = ({ onActionComplete }) => { event.preventDefault(); try { - await onCreateAccount(newAccountName || defaultAccountName); + await onCreateAccount(trimmedAccountName || defaultAccountName); onActionComplete(true); trackEvent({ category: MetaMetricsEventCategory.Accounts, diff --git a/ui/components/ui/editable-label/editable-label.js b/ui/components/ui/editable-label/editable-label.js index ec0f18df7..80a032450 100644 --- a/ui/components/ui/editable-label/editable-label.js +++ b/ui/components/ui/editable-label/editable-label.js @@ -38,7 +38,7 @@ export default class EditableLabel extends Component { return; } - await this.props.onSubmit(this.state.value); + await this.props.onSubmit(this.state.value.trim()); this.setState({ isEditing: false }); }