fix: expand click area for add new connection (#7395)

apache · Apr 28, 2024 · f51e718 · f51e718
1 parent 6ab1d29
commit f51e718
Showing 1 changed file with 8 additions and 3 deletions.
diff --git a/config-ui/src/routes/blueprint/detail/components/add-connection-dialog/index.tsx b/config-ui/src/routes/blueprint/detail/components/add-connection-dialog/index.tsx
@@ -68,7 +68,7 @@ export const AddConnectionDialog = ({ disabled = [], onCancel, onSubmit }: Props
 
   const options = useMemo(
     () =>
-      [{}].concat(
+      [{ value: '' }].concat(
         connections
           .filter((cs) => (disabled.length ? !disabled.includes(cs.unique) : true))
           .map((cs) => ({
@@ -110,7 +110,7 @@ export const AddConnectionDialog = ({ disabled = [], onCancel, onSubmit }: Props
               optionRender={(option, { index }) => {
                 if (index === 0) {
                   return (
-                    <Button size="small" type="link" icon={<PlusOutlined />} onClick={() => navigate('/connections')}>
+                    <Button size="small" type="link" icon={<PlusOutlined />}>
                       Add New Connection
                     </Button>
                   );
@@ -121,7 +121,12 @@ export const AddConnectionDialog = ({ disabled = [], onCancel, onSubmit }: Props
                   </Option>
                 );
               }}
-              onChange={(value) => setSelectedValue(value)}
+              onChange={(value) => {
+                if (!value) {
+                  navigate('/connections');
+                }
+                setSelectedValue(value);
+              }}
             />
           </Block>
           <Space style={{ display: 'flex', justifyContent: 'flex-end' }}>