feat: improve code

packages/client/src/acl/Configuration/MenuConfigure.tsx

@@ -4,10 +4,23 @@ import { useMenuItems } from '.';
 import { useAPIClient, useRequest } from '../../api-client';
 import { useRecord } from '../../record-provider';
 
+const findUids = (items) => {
+  if (!Array.isArray(items)) {
+    return [];
+  }
+  const uids = [];
+  for (const item of items) {
+    uids.push(item.uid);
+    uids.push(...findUids(item.children));
+  }
+  return uids;
+};
+
 export const MenuConfigure = () => {
   const record = useRecord();
   const api = useAPIClient();
   const items = useMenuItems();
+  const allUids = findUids(items);
   const [uids, setUids] = useState([]);
   const { loading, refresh } = useRequest(
     {
@@ -25,7 +38,7 @@ export const MenuConfigure = () => {
     },
   );
   const resource = api.resource('roles.menuUiSchemas', record.name);
-  const allChecked = items.length === uids.length;
+  const allChecked = allUids.length === uids.length;
   return (
     <Table
       loading={loading}
@@ -52,7 +65,7 @@ export const MenuConfigure = () => {
                     });
                   } else {
                     await resource.set({
-                      values: items.map((item) => item.uid),
+                      values: allUids,
                     });
                   }
                   refresh();
@@ -62,7 +75,8 @@ export const MenuConfigure = () => {
               允许访问
             </>
           ),
-          render: (checked, schema) => {
+          render: (_, schema) => {
+            const checked = uids.includes(schema.uid);
             return (
               <Checkbox
                 checked={checked}
@@ -86,10 +100,7 @@ export const MenuConfigure = () => {
           },
         },
       ]}
-      dataSource={items.map((item) => {
-        const accessible = uids.includes(item.uid);
-        return { ...item, accessible };
-      })}
+      dataSource={items}
     />
   );
 };

packages/client/src/schema-component/antd/action/Action.Modal.tsx

@@ -1,6 +1,7 @@
 import { css } from '@emotion/css';
 import { observer, RecursionField, useField, useFieldSchema } from '@formily/react';
 import { Modal } from 'antd';
+import classNames from 'classnames';
 import React from 'react';
 import { createPortal } from 'react-dom';
 import { useActionContext } from '.';
@@ -27,8 +28,18 @@ export const ActionModal: ComposedActionDrawer = observer((props) => {
           destroyOnClose
           visible={visible}
           onCancel={() => setVisible(false)}
+          className={classNames(
+            others.className,
+            css`
+              &.nb-action-popup {
+                .ant-modal-body {
+                  background: #f0f2f5;
+                }
+              }
+            `,
+          )}
           footer={
-            footerSchema && (
+            footerSchema ? (
               <div
                 className={css`
                   display: flex;
@@ -48,6 +59,8 @@ export const ActionModal: ComposedActionDrawer = observer((props) => {
                   }}
                 />
               </div>
+            ) : (
+              false
             )
           }
         >

packages/client/src/schema-component/antd/kanban/Kanban.Card.tsx

@@ -1,3 +1,4 @@
+import { css } from '@emotion/css';
 import { FormLayout } from '@formily/antd';
 import { observer, RecursionField, useFieldSchema } from '@formily/react';
 import { Card } from 'antd';
@@ -27,6 +28,18 @@ export const KanbanCard: any = observer((props: any) => {
         hoverable
         style={{ cursor: 'pointer', overflow: 'hidden' }}
         bodyStyle={{ paddingBottom: 0 }}
+        className={css`
+          .ant-description-input {
+            text-overflow: ellipsis;
+            width: 100%;
+            overflow: hidden;
+          }
+          .ant-description-textarea {
+            text-overflow: ellipsis;
+            width: 100%;
+            overflow: hidden;
+          }
+        `}
       >
         <SchemaComponentOptions components={{}}>
           <DndContext

packages/client/src/schema-component/antd/upload/Upload.tsx

@@ -123,40 +123,42 @@ Upload.Attachment = connect((props: UploadProps) => {
               </div>
             );
           })}
-          <div className={'ant-upload-list-picture-card-container'}>
-            <AntdUpload
-              {...useUploadProps({ ...props })}
-              disabled={disabled}
-              multiple={multiple}
-              listType={'picture-card'}
-              fileList={fileList}
-              onChange={(info) => {
-                setSync(false);
-                if (multiple) {
-                  if (info.file.status === 'done') {
-                    onChange(toValue(info.fileList));
+          {!disabled && (multiple || toArr(value).length < 1) && (
+            <div className={'ant-upload-list-picture-card-container'}>
+              <AntdUpload
+                {...useUploadProps({ ...props })}
+                disabled={disabled}
+                multiple={multiple}
+                listType={'picture-card'}
+                fileList={fileList}
+                onChange={(info) => {
+                  setSync(false);
+                  if (multiple) {
+                    if (info.file.status === 'done') {
+                      onChange(toValue(info.fileList));
+                    }
+                    setFileList(info.fileList.map(toItem));
+                  } else {
+                    if (info.file.status === 'done') {
+                      console.log('field.value', info.file?.response?.data);
+                      // TODO(BUG): object 的联动有问题，不响应，折中的办法先置空再赋值
+                      onChange(null);
+                      onChange(info.file?.response?.data);
+                    }
+                    setFileList([toItem(info.file)]);
                   }
-                  setFileList(info.fileList.map(toItem));
-                } else {
-                  if (info.file.status === 'done') {
-                    console.log('field.value', info.file?.response?.data);
-                    // TODO(BUG): object 的联动有问题，不响应，折中的办法先置空再赋值
-                    onChange(null);
-                    onChange(info.file?.response?.data);
-                  }
-                  setFileList([toItem(info.file)]);
-                }
-              }}
-              showUploadList={false}
-            >
-              {!disabled && (multiple || toArr(value).length < 1) && (
-                <span>
-                  <PlusOutlined />
-                  <br /> {t('Upload')}
-                </span>
-              )}
-            </AntdUpload>
-          </div>
+                }}
+                showUploadList={false}
+              >
+                {!disabled && (multiple || toArr(value).length < 1) && (
+                  <span>
+                    <PlusOutlined />
+                    <br /> {t('Upload')}
+                  </span>
+                )}
+              </AntdUpload>
+            </div>
+          )}
         </div>
       </div>
       {visible && (

packages/client/src/schema-component/common/sortable-item/SortableItem.tsx

@@ -80,7 +80,7 @@ export const DragHandler = (props) => {
         {...listeners}
         {...attributes}
       >
-        <span style={{ cursor: 'move', fontSize: 12 }}>{props.children}</span>
+        <span style={{ cursor: 'move', fontSize: 14 }}>{props.children}</span>
       </div>
     </div>
   );