diff --git a/web/app/components/app-sidebar/dataset-info/dropdown.tsx b/web/app/components/app-sidebar/dataset-info/dropdown.tsx index 8f3a25738a..a8a7c51ae8 100644 --- a/web/app/components/app-sidebar/dataset-info/dropdown.tsx +++ b/web/app/components/app-sidebar/dataset-info/dropdown.tsx @@ -89,9 +89,13 @@ const DropDown = ({ setConfirmMessage(isUsedByApp ? t('datasetUsedByApp', { ns: 'dataset' })! : t('deleteDatasetConfirmContent', { ns: 'dataset' })!) setShowConfirmDelete(true) } - catch (e: any) { - const res = await e.json() - toast(res?.message || 'Unknown error', { type: 'error' }) + catch (e: unknown) { + let message = 'Unknown error' + if (e instanceof Response) { + const res = await e.json() as { message?: string } + message = res?.message || message + } + toast(message, { type: 'error' }) } }, [dataset.id, t]) @@ -112,10 +116,15 @@ const DropDown = ({ open={open} onOpenChange={setOpen} > - }> - - - + + )} + > + - - + className={cn( + 'inline-flex h-4 w-4 shrink-0 cursor-pointer items-center justify-center border-0 bg-transparent p-0 focus-visible:ring-1 focus-visible:ring-components-input-border-hover focus-visible:outline-hidden', + className, )} - /> + > + + = ({ }, [activeTab, t]) const defaultTriggerElement = ( -
- -
+ + ) - const triggerElement = trigger ? trigger(open) : defaultTriggerElement + const triggerElement = trigger?.(open) const triggerElementProps = React.isValidElement(triggerElement) ? (triggerElement.props as { onClick?: MouseEventHandler @@ -216,15 +218,17 @@ const NodeSelector: FC = ({ const sideOffset = typeof offset === 'number' ? offset : (resolvedOffset?.mainAxis ?? 0) const alignOffset = typeof offset === 'number' ? 0 : (resolvedOffset?.crossAxis ?? 0) const nativeButton = asChild - && React.isValidElement(triggerElement) - && (typeof triggerElement.type !== 'string' || triggerElement.type === 'button') + ? React.isValidElement(triggerElement) && (typeof triggerElement.type !== 'string' || triggerElement.type === 'button') + : false return ( - + {trigger + ? + : defaultTriggerElement} = ({ ? variablePicker : ( @@ -344,6 +345,7 @@ const VarReferencePickerTrigger: FC = ({ return (