From d1d5416077232228b2c765f1c07c5515fd5c09f5 Mon Sep 17 00:00:00 2001 From: Sergey Petushkov Date: Fri, 30 Jun 2023 15:40:29 +0200 Subject: [PATCH] Revert "feat(compass-crud): add bloated docs insight to the document list COMPASS-6837 (#4566)" This reverts commit bfb9cbb4231256a42735e555cf322ba812f40577. --- package-lock.json | 2 - .../document-list/document-actions-group.tsx | 43 +-------- .../src/components/signal-popover.tsx | 25 ++--- packages/compass-crud/package.json | 1 - .../src/components/editable-document.tsx | 18 +--- .../src/components/readonly-document.tsx | 17 +--- .../compass-crud/src/stores/crud-store.ts | 95 ++++--------------- .../compass-field-store/src/stores/store.js | 8 +- .../src/stores/store.spec.js | 4 +- packages/hadron-document/src/document.ts | 1 - 10 files changed, 44 insertions(+), 170 deletions(-) diff --git a/package-lock.json b/package-lock.json index ad0b38027d9..fdf5fafa52e 100644 --- a/package-lock.json +++ b/package-lock.json @@ -42338,7 +42338,6 @@ "react-dom": "^17.0.2", "reflux": "^0.4.1", "reflux-state-mixin": "github:mongodb-js/reflux-state-mixin", - "semver": "^7.5.2", "sinon": "^8.1.1" }, "peerDependencies": { @@ -57453,7 +57452,6 @@ "react-dom": "^17.0.2", "reflux": "^0.4.1", "reflux-state-mixin": "github:mongodb-js/reflux-state-mixin", - "semver": "^7.5.2", "sinon": "^8.1.1" } }, diff --git a/packages/compass-components/src/components/document-list/document-actions-group.tsx b/packages/compass-components/src/components/document-list/document-actions-group.tsx index 12792ed52b9..0b35de05910 100644 --- a/packages/compass-components/src/components/document-list/document-actions-group.tsx +++ b/packages/compass-components/src/components/document-list/document-actions-group.tsx @@ -1,15 +1,12 @@ import React, { useEffect, useRef, useState } from 'react'; -import { css, cx } from '@leafygreen-ui/emotion'; +import { css } from '@leafygreen-ui/emotion'; import { spacing } from '@leafygreen-ui/tokens'; import { Button, Icon } from '../leafygreen'; import { Tooltip } from '../tooltip'; -import type { Signal } from '../signal-popover'; -import { SignalPopover } from '../signal-popover'; const actionsGroupContainer = css({ position: 'absolute', display: 'flex', - alignItems: 'center', gap: spacing[2], width: '100%', top: spacing[2] + spacing[1], @@ -25,24 +22,6 @@ const actionsGroupItem = css({ const actionsGroupItemSeparator = css({ flex: '1 0 auto', - pointerEvents: 'none', -}); - -const actionsGroupIdle = css({ - '& > button': { - display: 'none', - }, -}); - -const actionsGroupHovered = css({ - '& > button': { - display: 'block', - }, -}); - -// Insight icon is always visible, even when action buttons are not -const actionsGroupSignalPopover = css({ - display: 'block !important', }); function useElementParentHoverState( @@ -80,7 +59,6 @@ const DocumentActionsGroup: React.FunctionComponent< onClone?: () => void; onRemove?: () => void; onlyShowOnHover?: boolean; - insights?: Signal | Signal[]; } & ( | { onExpand?: never; expanded?: never } | { onExpand: () => void; expanded: boolean } @@ -93,13 +71,10 @@ const DocumentActionsGroup: React.FunctionComponent< onExpand, expanded, onlyShowOnHover = true, - insights, }) => { - const [signalOpened, setSignalOpened] = useState(false); const conatinerRef = useRef(null); const isHovered = useElementParentHoverState(conatinerRef); const [showCopyButtonTooltip, setShowCopyButtonTooltip] = useState(false); - const isActive = isHovered || signalOpened; useEffect(() => { if (showCopyButtonTooltip === true) { @@ -115,10 +90,10 @@ const DocumentActionsGroup: React.FunctionComponent< return (
{onExpand && ( )} - {insights && ( -
- -
- )} {onEdit && (