do not allow account names with just spaces

MetaMask · Jul 25, 2023 · 9c6e8db · 9c6e8db
1 parent 12fc72f
commit 9c6e8db
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git 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 === '' ? defaultAccountName : 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
@@ -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 });
   }