diff --git a/frontend/src/component/feature/FeatureView/FeatureOverview/FeatureOverviewEnvironments/FeatureOverviewEnvironment/EnvironmentAccordionBody/EnvironmentAccordionBody.tsx b/frontend/src/component/feature/FeatureView/FeatureOverview/FeatureOverviewEnvironments/FeatureOverviewEnvironment/EnvironmentAccordionBody/EnvironmentAccordionBody.tsx index 148d2b7f15..1468fdbeb6 100644 --- a/frontend/src/component/feature/FeatureView/FeatureOverview/FeatureOverviewEnvironments/FeatureOverviewEnvironment/EnvironmentAccordionBody/EnvironmentAccordionBody.tsx +++ b/frontend/src/component/feature/FeatureView/FeatureOverview/FeatureOverviewEnvironments/FeatureOverviewEnvironment/EnvironmentAccordionBody/EnvironmentAccordionBody.tsx @@ -74,10 +74,12 @@ const AlertContainer = styled('div')(({ theme }) => ({ }, })); -const StyledStrategySeparator = styled(StrategySeparator)({ - [`&:has(+ * .${disabledStrategyClassName})`]: { - filter: 'grayscale(1)', - }, +// todo: consider exporting this into a shared thing or move it into the separator itself (either as a disabled prop or using the css here) +export const StyledStrategySeparator = styled(StrategySeparator)({ + [`&:has(+ * .${disabledStrategyClassName}, + .${disabledStrategyClassName})`]: + { + filter: 'grayscale(1)', + }, }); export const EnvironmentAccordionBody = ({ diff --git a/frontend/src/component/playground/Playground/PlaygroundResultsTable/FeatureResultInfoPopoverCell/FeatureStrategyList/StrategyList/PlaygroundResultStrategyLists.tsx b/frontend/src/component/playground/Playground/PlaygroundResultsTable/FeatureResultInfoPopoverCell/FeatureStrategyList/StrategyList/PlaygroundResultStrategyLists.tsx index 4149209fab..d3faab4db8 100644 --- a/frontend/src/component/playground/Playground/PlaygroundResultsTable/FeatureResultInfoPopoverCell/FeatureStrategyList/StrategyList/PlaygroundResultStrategyLists.tsx +++ b/frontend/src/component/playground/Playground/PlaygroundResultsTable/FeatureResultInfoPopoverCell/FeatureStrategyList/StrategyList/PlaygroundResultStrategyLists.tsx @@ -1,4 +1,4 @@ -import { Alert, styled } from '@mui/material'; +import { styled } from '@mui/material'; import type { PlaygroundStrategySchema, PlaygroundRequestSchema, @@ -6,29 +6,10 @@ import type { import { StyledContentList, StyledListItem, + StyledStrategySeparator, } from 'component/feature/FeatureView/FeatureOverview/FeatureOverviewEnvironments/FeatureOverviewEnvironment/EnvironmentAccordionBody/EnvironmentAccordionBody'; -import { StrategySeparator } from 'component/common/StrategySeparator/StrategySeparator'; import { FeatureStrategyItem } from './StrategyItem/FeatureStrategyItem'; -const StyledAlertWrapper = styled('div')(({ theme }) => ({ - display: 'flex', - padding: `0, 4px`, - flexDirection: 'column', - borderRadius: theme.shape.borderRadiusMedium, - border: `1px solid ${theme.palette.warning.border}`, -})); - -const StyledListWrapper = styled('div')(({ theme }) => ({ - padding: theme.spacing(1, 0.5), -})); - -const StyledAlert = styled(Alert)(({ theme }) => ({ - border: '0!important', - borderBottomLeftRadius: 0, - borderBottomRightRadius: 0, - borderBottom: `1px solid ${theme.palette.warning.border}!important`, -})); - interface PlaygroundResultStrategyListProps { strategies: PlaygroundStrategySchema[]; input?: PlaygroundRequestSchema; @@ -85,7 +66,7 @@ export const PlaygroundResultStrategyLists = ({ {strategies?.map((strategy, index) => ( - {index > 0 ? : ''} + {index > 0 ? : ''} { const { result } = strategy; - const theme = useTheme(); const label = result.evaluationStatus === 'incomplete' || result.evaluationStatus === 'unevaluated'