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 c09eb9e79e..6d16d97818 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 @@ -238,8 +238,8 @@ export const EnvironmentAccordionBody = ({ ); }; - const Strategies = () => { - return strategies.length < 50 || !manyStrategiesPagination ? ( + const strategyList = + strategies.length < 50 || !manyStrategiesPagination ? ( {strategies.map((strategy, index) => ( @@ -287,7 +287,6 @@ export const EnvironmentAccordionBody = ({ /> ); - }; return ( @@ -305,12 +304,12 @@ export const EnvironmentAccordionBody = ({ {strategies.length > 0 ? (
  • - + {strategyList}
  • ) : null} ) : strategies.length > 0 ? ( - + strategyList ) : null}
    diff --git a/frontend/src/component/feature/FeatureView/FeatureOverview/FeatureOverviewEnvironments/FeatureOverviewEnvironment/EnvironmentAccordionBody/StrategyDraggableItem/ProjectEnvironmentStrategyDraggableItem.tsx b/frontend/src/component/feature/FeatureView/FeatureOverview/FeatureOverviewEnvironments/FeatureOverviewEnvironment/EnvironmentAccordionBody/StrategyDraggableItem/ProjectEnvironmentStrategyDraggableItem.tsx index d68f6b2332..49dad7698a 100644 --- a/frontend/src/component/feature/FeatureView/FeatureOverview/FeatureOverviewEnvironments/FeatureOverviewEnvironment/EnvironmentAccordionBody/StrategyDraggableItem/ProjectEnvironmentStrategyDraggableItem.tsx +++ b/frontend/src/component/feature/FeatureView/FeatureOverview/FeatureOverviewEnvironments/FeatureOverviewEnvironment/EnvironmentAccordionBody/StrategyDraggableItem/ProjectEnvironmentStrategyDraggableItem.tsx @@ -85,6 +85,7 @@ export const ProjectEnvironmentStrategyDraggableItem = ({ onDragStartRef={onDragStartRef} onDragOver={onDragOver} index={index} + isDragging={isDragging} headerItemsRight={ <> {draftChange && !isSmallScreen ? ( diff --git a/frontend/src/component/releases/ReleasePlanTemplate/TemplateForm/MilestoneList/MilestoneCard/MilestoneCard.tsx b/frontend/src/component/releases/ReleasePlanTemplate/TemplateForm/MilestoneList/MilestoneCard/MilestoneCard.tsx index 6c3512a8e1..2f9b48bcd7 100644 --- a/frontend/src/component/releases/ReleasePlanTemplate/TemplateForm/MilestoneList/MilestoneCard/MilestoneCard.tsx +++ b/frontend/src/component/releases/ReleasePlanTemplate/TemplateForm/MilestoneList/MilestoneCard/MilestoneCard.tsx @@ -115,6 +115,7 @@ const StyledAccordionFooter = styled('div')(({ theme }) => ({ justifyContent: 'flex-end', gap: theme.spacing(3), backgroundColor: 'inherit', + borderTop: `1px solid ${theme.palette.divider}`, })); const StyledIconButton = styled(IconButton)(({ theme }) => ({ @@ -324,12 +325,11 @@ export const MilestoneCard = ({ height: ref.current?.offsetHeight || 0, }); - if (ref?.current) { - event.dataTransfer.effectAllowed = 'move'; - event.dataTransfer.setData('text/html', ref.current.outerHTML); - event.dataTransfer.setDragImage(ref.current, 20, 20); - } + event.dataTransfer.effectAllowed = 'move'; + event.dataTransfer.setData('text/html', ref.current.outerHTML); + event.dataTransfer.setDragImage(ref.current, 20, 20); }; + const onStrategyDragEnd = () => { setDragItem(null); onReOrderStrategies();