From f71b584817f63a6d244d95fa2c8dd61d2a53d0dd Mon Sep 17 00:00:00 2001
From: Aries <urchinzhou@gmail.com>
Date: Thu, 19 Jun 2025 09:53:11 +0800
Subject: [PATCH] remove global hidden columns in details setting (#7951)

Co-authored-by: zhouwenxuan <aries@Mac.local>
---
 .../metadata/components/metadata-details/settings-icon.js  | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/frontend/src/metadata/components/metadata-details/settings-icon.js b/frontend/src/metadata/components/metadata-details/settings-icon.js
index 84cd5ac912..ffaa101dae 100644
--- a/frontend/src/metadata/components/metadata-details/settings-icon.js
+++ b/frontend/src/metadata/components/metadata-details/settings-icon.js
@@ -7,9 +7,10 @@ import { useMetadataStatus } from '../../../hooks';
 const SettingsIcon = () => {
   const [isShowSetter, setShowSetter] = useState(false);
 
-  const { enableMetadata } = useMetadataStatus();
+  const { enableMetadata, globalHiddenColumns } = useMetadataStatus();
   const { modifyColumnOrder, modifyHiddenColumns, record, columns, canModifyDetails } = useMetadataDetails();
-  const hiddenColumns = useMemo(() => columns.filter(c => !c.shown).map(c => c.key), [columns]);
+  const validColumns = useMemo(() => columns.filter(c => !globalHiddenColumns.includes(c.key)), [columns, globalHiddenColumns]);
+  const hiddenColumns = useMemo(() => validColumns.filter(c => !c.shown).map(c => c.key), [validColumns]);
 
   const onSetterToggle = useCallback(() => {
     setShowSetter(!isShowSetter);
@@ -29,7 +30,7 @@ const SettingsIcon = () => {
           hiddenColumns={hiddenColumns}
           target={target}
           placement="bottom-end"
-          columns={columns}
+          columns={validColumns}
           hidePopover={onSetterToggle}
           onChange={modifyHiddenColumns}
           modifyColumnOrder={modifyColumnOrder}