From d743082d83ea43c097e34b1788933187eaf72346 Mon Sep 17 00:00:00 2001 From: Cats Juice Date: Tue, 24 Jun 2025 10:17:57 +0800 Subject: [PATCH] fix(mobile): change mobile docs display config and prevent opening doc when close menu (#12903) ## Summary by CodeRabbit - **Improvements** - Updated document explorer to hide quick favorite and document icons, and display more detailed system properties for each document. - Enhanced link behavior in the workbench to prevent unintended clicks from triggering navigation. --- .../frontend/core/src/mobile/pages/workspace/all.tsx | 9 +++++++-- .../core/src/modules/workbench/view/workbench-link.tsx | 5 +++++ 2 files changed, 12 insertions(+), 2 deletions(-) diff --git a/packages/frontend/core/src/mobile/pages/workspace/all.tsx b/packages/frontend/core/src/mobile/pages/workspace/all.tsx index 9cf6b20aa..a4bda72ff 100644 --- a/packages/frontend/core/src/mobile/pages/workspace/all.tsx +++ b/packages/frontend/core/src/mobile/pages/workspace/all.tsx @@ -15,8 +15,13 @@ import { AllDocsHeader } from '../../views'; const AllDocs = () => { const [explorerContextValue] = useState(() => createDocExplorerContext({ - quickFavorite: true, - displayProperties: ['createdAt', 'updatedAt', 'tags'], + quickFavorite: false, + showDocIcon: false, + displayProperties: [ + 'system:createdAt', + 'system:updatedAt', + 'system:tags', + ], view: 'masonry', showDragHandle: false, groupBy: undefined, diff --git a/packages/frontend/core/src/modules/workbench/view/workbench-link.tsx b/packages/frontend/core/src/modules/workbench/view/workbench-link.tsx index f1a42ab8c..88b7a0a99 100644 --- a/packages/frontend/core/src/modules/workbench/view/workbench-link.tsx +++ b/packages/frontend/core/src/modules/workbench/view/workbench-link.tsx @@ -61,6 +61,11 @@ export const WorkbenchLink = forwardRef( const link = basename + stringTo; const handleClick = useAsyncCallback( async (event: React.MouseEvent) => { + // a trick to prevent click from modal overlay propagation + if (!event.currentTarget.contains(event.target as Node)) { + return; + } + onClick?.(event); if (event.defaultPrevented) { return;