Disable space fields whilst their form is busy

matrix-org · May 12, 2021 · 2c89be3 · 2c89be3
1 parent 5d02e44
commit 2c89be3
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 4 deletions.
diff --git a/src/components/structures/SpaceRoomView.tsx b/src/components/structures/SpaceRoomView.tsx
@@ -451,6 +451,7 @@ const SpaceSetupFirstRooms = ({ space, title, description, onFinished }) => {
             value={roomNames[i]}
             onChange={ev => setRoomName(i, ev.target.value)}
             autoFocus={i === 2}
+            disabled={busy}
         />;
     });
 
@@ -658,6 +659,7 @@ const SpaceSetupPrivateInvite = ({ space, onFinished }) => {
             ref={fieldRefs[i]}
             onValidate={validateEmailRules}
             autoFocus={i === 0}
+            disabled={busy}
         />;
     });
 

diff --git a/src/components/views/dialogs/SpaceSettingsDialog.tsx b/src/components/views/dialogs/SpaceSettingsDialog.tsx
@@ -116,13 +116,13 @@ const SpaceSettingsDialog: React.FC<IProps> = ({ matrixClient: cli, space, onFin
 
             <SpaceBasicSettings
                 avatarUrl={avatarUrlForRoom(space, 80, 80, "crop")}
-                avatarDisabled={!canSetAvatar}
+                avatarDisabled={busy || !canSetAvatar}
                 setAvatar={setNewAvatar}
                 name={name}
-                nameDisabled={!canSetName}
+                nameDisabled={busy || !canSetName}
                 setName={setName}
                 topic={topic}
-                topicDisabled={!canSetTopic}
+                topicDisabled={busy || !canSetTopic}
                 setTopic={setTopic}
             />
 

diff --git a/src/components/views/spaces/SpaceCreateMenu.tsx b/src/components/views/spaces/SpaceCreateMenu.tsx
@@ -178,7 +178,7 @@ const SpaceCreateMenu = ({ onFinished }) => {
             </p>
 
             <form className="mx_SpaceBasicSettings" onSubmit={onSpaceCreateClick}>
-                <SpaceAvatar setAvatar={setAvatar} />
+                <SpaceAvatar setAvatar={setAvatar} avatarDisabled={busy} />
 
                 <Field
                     name="spaceName"
@@ -188,6 +188,7 @@ const SpaceCreateMenu = ({ onFinished }) => {
                     onChange={ev => setName(ev.target.value)}
                     ref={spaceNameField}
                     onValidate={spaceNameValidator}
+                    disabled={busy}
                 />
 
                 <Field
@@ -197,6 +198,7 @@ const SpaceCreateMenu = ({ onFinished }) => {
                     value={topic}
                     onChange={ev => setTopic(ev.target.value)}
                     rows={3}
+                    disabled={busy}
                 />
             </form>