PrivateCoffee/element-web
4
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 2

Fix create subspace dialog not working (#10652)

Browse source
This commit is contained in:
Michael Telatynski 2023-04-18 17:00:55 +01:00 • committed by GitHub
parent 83359f84d1
commit 0db40e327c
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 7 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

10
src/components/views/dialogs/CreateSubspaceDialog.tsx
View file

@ -56,18 +56,22 @@ const CreateSubspaceDialog: React.FC<IProps> = ({ space, onAddExistingSpaceClick
const onCreateSubspaceClick = async (e: ButtonEvent): Promise<void> => { const onCreateSubspaceClick = async (e: ButtonEvent): Promise<void> => {
e.preventDefault(); e.preventDefault();
if (busy || !spaceNameField.current || !spaceAliasField.current) return; if (busy) return;
setBusy(true); setBusy(true);
// require & validate the space name field // require & validate the space name field
if (!(await spaceNameField.current.validate({ allowEmpty: false }))) { if (spaceNameField.current && !(await spaceNameField.current.validate({ allowEmpty: false }))) {
spaceNameField.current.focus(); spaceNameField.current.focus();
spaceNameField.current.validate({ allowEmpty: false, focused: true }); spaceNameField.current.validate({ allowEmpty: false, focused: true });
setBusy(false); setBusy(false);
return; return;
} }
// validate the space name alias field but do not require it // validate the space name alias field but do not require it
if (joinRule === JoinRule.Public && !(await spaceAliasField.current.validate({ allowEmpty: true }))) { if (
spaceAliasField.current &&
joinRule === JoinRule.Public &&
(await spaceAliasField.current.validate({ allowEmpty: true }))
) {
spaceAliasField.current.focus(); spaceAliasField.current.focus();
spaceAliasField.current.validate({ allowEmpty: true, focused: true }); spaceAliasField.current.validate({ allowEmpty: true, focused: true });
setBusy(false); setBusy(false);