estel/dify
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

fix(ui): temporarily remove TagSelector from app sidebar per design review (#23329)

Browse Source
This commit is contained in:
yyh
2025-08-03 22:30:21 +08:00
committed by GitHub
parent 90373c7165
commit dedd5f571c
1 changed files with 3 additions and 37 deletions
Download Patch File Download Diff File Expand all files Collapse all files

40
web/app/components/app-sidebar/app-info.tsx
View File

@@ -1,7 +1,7 @@
import { useTranslation } from 'react-i18next' import { useTranslation } from 'react-i18next'
import { useRouter } from 'next/navigation' import { useRouter } from 'next/navigation'
import { useContext } from 'use-context-selector' import { useContext } from 'use-context-selector'
import React, { useCallback, useEffect, useState } from 'react' import React, { useCallback, useState } from 'react'
import { import {
RiDeleteBinLine, RiDeleteBinLine,
RiEditLine, RiEditLine,
@@ -18,8 +18,6 @@ import { ToastContext } from '@/app/components/base/toast'
import { useAppContext } from '@/context/app-context' import { useAppContext } from '@/context/app-context'
import { useProviderContext } from '@/context/provider-context' import { useProviderContext } from '@/context/provider-context'
import { copyApp, deleteApp, exportAppConfig, updateAppInfo } from '@/service/apps' import { copyApp, deleteApp, exportAppConfig, updateAppInfo } from '@/service/apps'
import type { Tag } from '@/app/components/base/tag-management/constant'
import TagSelector from '@/app/components/base/tag-management/selector'
import type { DuplicateAppModalProps } from '@/app/components/app/duplicate-modal' import type { DuplicateAppModalProps } from '@/app/components/app/duplicate-modal'
import type { CreateAppModalProps } from '@/app/components/explore/create-app-modal' import type { CreateAppModalProps } from '@/app/components/explore/create-app-modal'
import { NEED_REFRESH_APP_LIST_KEY } from '@/config' import { NEED_REFRESH_APP_LIST_KEY } from '@/config'
@@ -75,11 +73,6 @@ const AppInfo = ({ expand, onlyShowDetail = false, openState = false, onDetailEx
const [showImportDSLModal, setShowImportDSLModal] = useState<boolean>(false) const [showImportDSLModal, setShowImportDSLModal] = useState<boolean>(false)
const [secretEnvList, setSecretEnvList] = useState<EnvironmentVariable[]>([]) const [secretEnvList, setSecretEnvList] = useState<EnvironmentVariable[]>([])
const [tags, setTags] = useState<Tag[]>(appDetail?.tags || [])
useEffect(() => {
setTags(appDetail?.tags || [])
}, [appDetail?.tags])
const onEdit: CreateAppModalProps['onConfirm'] = useCallback(async ({ const onEdit: CreateAppModalProps['onConfirm'] = useCallback(async ({
name, name,
icon_type, icon_type,
@@ -310,35 +303,8 @@ const AppInfo = ({ expand, onlyShowDetail = false, openState = false, onDetailEx
imageUrl={appDetail.icon_url} imageUrl={appDetail.icon_url}
/> />
<div className='flex w-full grow flex-col items-start justify-center'> <div className='flex w-full grow flex-col items-start justify-center'>
<div className='flex w-full items-center justify-between'> <div className='system-md-semibold w-full truncate text-text-secondary'>{appDetail.name}</div>
<div className='flex min-w-0 flex-1 flex-col'> <div className='system-2xs-medium-uppercase text-text-tertiary'>{appDetail.mode === 'advanced-chat' ? t('app.types.advanced') : appDetail.mode === 'agent-chat' ? t('app.types.agent') : appDetail.mode === 'chat' ? t('app.types.chatbot') : appDetail.mode === 'completion' ? t('app.types.completion') : t('app.types.workflow')}</div>
<div className='flex items-center gap-2'>
<div className='system-md-semibold truncate text-text-secondary'>{appDetail.name}</div>
{isCurrentWorkspaceEditor && (
<div className='flex w-0 grow items-center' onClick={(e) => {
e.stopPropagation()
e.preventDefault()
}}>
<div className='w-full'>
<TagSelector
position='br'
type='app'
targetID={appDetail.id}
value={tags.map(tag => tag.id)}
selectedTags={tags}
onCacheUpdate={setTags}
onChange={() => {
// Optional: could trigger a refresh if needed
}}
minWidth='true'
/>
</div>
</div>
)}
</div>
<div className='system-2xs-medium-uppercase whitespace-nowrap text-text-tertiary'>{appDetail.mode === 'advanced-chat' ? t('app.types.advanced') : appDetail.mode === 'agent-chat' ? t('app.types.agent') : appDetail.mode === 'chat' ? t('app.types.chatbot') : appDetail.mode === 'completion' ? t('app.types.completion') : t('app.types.workflow')}</div>
</div>
</div>
</div> </div>
</div> </div>
{/* description */} {/* description */}