diff --git a/src/library-authoring/containers/ContainerInfo.tsx b/src/library-authoring/containers/ContainerInfo.tsx index 2b36a0764d..243a12278e 100644 --- a/src/library-authoring/containers/ContainerInfo.tsx +++ b/src/library-authoring/containers/ContainerInfo.tsx @@ -2,13 +2,13 @@ import { FormattedMessage, useIntl } from '@edx/frontend-platform/i18n'; import { Button, Stack, - Tab, - Tabs, Dropdown, Icon, IconButton, useToggle, } from '@openedx/paragon'; +// eslint-disable-next-line import/no-extraneous-dependencies +import { Tab, Nav } from 'react-bootstrap'; import React, { useCallback } from 'react'; import { Link } from 'react-router-dom'; import { MoreVert } from '@openedx/paragon/icons'; @@ -39,7 +39,6 @@ type ContainerPreviewProps = { const ContainerMenu = ({ containerId }: ContainerPreviewProps) => { const intl = useIntl(); - const [isConfirmingDelete, confirmDelete, cancelDelete] = useToggle(false); return ( @@ -159,23 +158,21 @@ const ContainerInfo = () => { sidebarTab && isContainerInfoTab(sidebarTab) ) ? sidebarTab : defaultContainerTab; - /* istanbul ignore next */ const handleTabChange = (newTab: ContainerInfoTab) => { resetSidebarAction(); setSidebarTab(newTab); }; - const renderTab = useCallback((infoTab: ContainerInfoTab, title: string, component?: React.ReactNode) => { + const renderTab = useCallback((infoTab: ContainerInfoTab, title: string) => { if (hiddenTabs.includes(infoTab)) { - // For some reason, returning anything other than empty list breaks the tab style - return []; + return null; } return ( - - {component} - + + {title} + ); - }, [hiddenTabs, defaultContainerTab, containerId]); + }, [hiddenTabs]); if (!container || !containerId || !containerType) { return null; @@ -188,34 +185,50 @@ const ContainerInfo = () => { containerType={containerType} hasUnpublishedChanges={container.hasUnpublishedChanges} /> - handleTabChange(k as ContainerInfoTab)} + mountOnEnter + unmountOnExit > - {renderTab( - CONTAINER_INFO_TABS.Preview, - intl.formatMessage(messages.previewTabTitle), - , - )} - {renderTab( - CONTAINER_INFO_TABS.Manage, - intl.formatMessage(messages.manageTabTitle), - , - )} - {renderTab( - CONTAINER_INFO_TABS.Usage, - intl.formatMessage(messages.usageTabTitle), - , - )} - {renderTab( - CONTAINER_INFO_TABS.Settings, - intl.formatMessage(messages.settingsTabTitle), - // TODO: container settings component - )} - + + + + + + + + + + + + + + {/* TODO: Container settings component */} + + + ); };