mirror of
https://github.com/nocobase/nocobase
synced 2024-11-16 01:17:43 +00:00
perf(PageContent): improve performance
This commit is contained in:
parent
bab0661d5c
commit
5965852bf6
@ -114,13 +114,6 @@ export const useStyles = genStyleHook('nb-page', (token) => {
|
||||
},
|
||||
},
|
||||
|
||||
'.pageWithFixedBlockCss': {
|
||||
height: '100%',
|
||||
'> .nb-grid:not(:last-child)': {
|
||||
'> .nb-schema-initializer-button': { display: 'none' },
|
||||
},
|
||||
},
|
||||
|
||||
'.nb-page-wrapper': {
|
||||
padding: `${token.paddingPageVertical}px`,
|
||||
paddingBottom: 0,
|
||||
|
@ -22,7 +22,6 @@ import { FormDialog } from '..';
|
||||
import { antTableCell } from '../../../acl/style';
|
||||
import { useRequest } from '../../../api-client';
|
||||
import { useNavigateNoUpdate } from '../../../application/CustomRouterContextProvider';
|
||||
import { useAppSpin } from '../../../application/hooks/useAppSpin';
|
||||
import { useRouterBasename } from '../../../application/hooks/useRouterBasename';
|
||||
import { useDocumentTitle } from '../../../document-title';
|
||||
import { useGlobalTheme } from '../../../global-theme';
|
||||
@ -51,7 +50,7 @@ export const Page = (props) => {
|
||||
const options = useContext(SchemaOptionsContext);
|
||||
const navigate = useNavigateNoUpdate();
|
||||
const [searchParams] = useSearchParams();
|
||||
const [loading, setLoading] = useState(false);
|
||||
const loading = false;
|
||||
const activeKey = useMemo(
|
||||
// 处理 searchParams 是为了兼容旧版的 tab 参数
|
||||
() => tabUid || searchParams.get('tab') || Object.keys(fieldSchema.properties || {}).shift(),
|
||||
@ -78,11 +77,7 @@ export const Page = (props) => {
|
||||
marginRight: token.paddingPageHorizontal - token.paddingLG,
|
||||
}}
|
||||
onChange={(activeKey) => {
|
||||
setLoading(true);
|
||||
navigateToTab({ activeKey, navigate, basename: basenameOfCurrentRouter });
|
||||
setTimeout(() => {
|
||||
setLoading(false);
|
||||
}, 50);
|
||||
}}
|
||||
tabBarExtraContent={
|
||||
dn.designable && (
|
||||
@ -236,20 +231,20 @@ const PageContent = memo(
|
||||
fieldSchema: Schema<any, any, any, any, any, any, any, any, any>;
|
||||
activeKey: string;
|
||||
}) => {
|
||||
const { render } = useAppSpin();
|
||||
// const { render } = useAppSpin();
|
||||
|
||||
if (loading) {
|
||||
return render();
|
||||
}
|
||||
// if (loading) {
|
||||
// return render();
|
||||
// }g
|
||||
|
||||
return (
|
||||
<>
|
||||
{!disablePageHeader && enablePageTabs ? (
|
||||
fieldSchema.mapProperties((schema) => {
|
||||
if (!disablePageHeader && enablePageTabs) {
|
||||
const result = fieldSchema
|
||||
.mapProperties((schema) => {
|
||||
if (schema.name !== activeKey) return null;
|
||||
|
||||
return (
|
||||
<SchemaComponent
|
||||
key={schema.name}
|
||||
distributed
|
||||
schema={
|
||||
new Schema({
|
||||
@ -261,13 +256,16 @@ const PageContent = memo(
|
||||
/>
|
||||
);
|
||||
})
|
||||
) : (
|
||||
<div className={classNames(`pageWithFixedBlockCss nb-page-content`, className1)}>
|
||||
.filter(Boolean);
|
||||
|
||||
return result[0];
|
||||
} else {
|
||||
return (
|
||||
<div className={className1}>
|
||||
<SchemaComponent schema={fieldSchema} distributed />
|
||||
</div>
|
||||
)}
|
||||
</>
|
||||
);
|
||||
}
|
||||
},
|
||||
);
|
||||
PageContent.displayName = 'PageContent';
|
||||
|
@ -71,12 +71,16 @@ const RecursionSchemaComponent = memo((props: ISchemaFieldProps & SchemaComponen
|
||||
);
|
||||
});
|
||||
|
||||
RecursionSchemaComponent.displayName = 'RecursionSchemaComponent';
|
||||
|
||||
const MemoizedSchemaComponent = memo((props: ISchemaFieldProps & SchemaComponentOnChange & DistributedProps) => {
|
||||
const { schema, ...others } = props;
|
||||
const s = useMemoizedSchema(schema);
|
||||
return <RecursionSchemaComponent {...others} schema={s} />;
|
||||
});
|
||||
|
||||
MemoizedSchemaComponent.displayName = 'MemoizedSchemaComponent';
|
||||
|
||||
export const SchemaComponent = memo(
|
||||
(
|
||||
props: (ISchemaFieldProps | IRecursionFieldProps) & { memoized?: boolean } & SchemaComponentOnChange &
|
||||
@ -89,3 +93,5 @@ export const SchemaComponent = memo(
|
||||
return <RecursionSchemaComponent {...others} />;
|
||||
},
|
||||
);
|
||||
|
||||
SchemaComponent.displayName = 'SchemaComponent';
|
||||
|
Loading…
Reference in New Issue
Block a user