Merge pull request #158 from czhen-zilliz/refine-advanced-search

fix: Refine advanced search

client/src/components/advancedSearch/Dialog.tsx

@@ -42,6 +42,8 @@ const AdvancedDialog = (props: DialogProps) => {
     // eslint-disable-next-line react-hooks/exhaustive-deps
   }, []);
 
+  const shouldSowPlaceholder: boolean = !isLegal || !filterExpression;
+
   return (
     <>
       <Dialog
@@ -65,17 +67,17 @@ const AdvancedDialog = (props: DialogProps) => {
             <CloseIcon />
           </IconButton>
         </DialogTitle>
+        <div
+          className={`${classes.expResult} ${
+            shouldSowPlaceholder && 'disable-exp'
+          }`}
+        >
+          {`${shouldSowPlaceholder ? 'Filter Expression' : filterExpression}`}
+          {!shouldSowPlaceholder && (
+            <CopyBtn label="copy expression" value={filterExpression} />
+          )}
+        </div>
         <DialogContent>
-          <div
-            className={`${classes.expResult} ${
-              !isLegal && 'disable-exp'
-            } testcopy`}
-          >
-            {`${isLegal ? filterExpression : 'Filter Expression'}`}
-            {isLegal && (
-              <CopyBtn label="copy expression" value={filterExpression} />
-            )}
-          </div>
           <div className={classes.expWrapper}>
             <ConditionGroup
               fields={fields}
@@ -198,9 +200,8 @@ const useStyles = makeStyles((theme: Theme) =>
       justifyContent: 'space-between',
       alignItems: 'center',
       minHeight: '40px',
-      marginBottom: '16px',
+      margin: '8px 32px',
       padding: '0 16px',
-      fontFamily: 'Source Code Pro',
       fontStyle: 'normal',
       fontWeight: 'normal',
       fontSize: '16px',