Skip to content

Commit 828ade6

Browse files
committed
Fix Project Overlays
1 parent 200b961 commit 828ade6

2 files changed

Lines changed: 10 additions & 7 deletions

File tree

src/frontend/src/pages/GanttPage/GanttChart/GanttChartComponents/GanttTaskBar/GanttTaskBarDisplay.tsx

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -48,7 +48,7 @@ const GanttTaskBarDisplay = <T,>({
4848
highlightTaskComparator
4949
}: GanttTaskBarDisplayProps<T>) => {
5050
const theme = useTheme();
51-
const hasChildren = task.children.length > 0;
51+
const hasOverlays = task.overlays.length > 0;
5252

5353
const ganttTaskBarHoverDetectionBoxStyles: CSSProperties = {
5454
gridColumnStart: getStartCol(task.start),
@@ -69,10 +69,10 @@ const GanttTaskBarDisplay = <T,>({
6969
gridColumnEnd: getEndCol(task.end),
7070
display: 'flex',
7171
alignItems: 'center',
72-
marginTop: hasChildren ? '-10px' : undefined,
73-
marginBottom: hasChildren ? '-10px' : undefined,
72+
marginTop: hasOverlays ? '-10px' : undefined,
73+
marginBottom: hasOverlays ? '-10px' : undefined,
7474
cursor: 'pointer',
75-
width: hasChildren ? 'fit-content' : '100%'
75+
width: hasOverlays ? 'fit-content' : '100%'
7676
};
7777

7878
const ganttTaskBarChildOverlayStyles = (child: GanttTask<T>): CSSProperties => {
@@ -174,7 +174,7 @@ const GanttTaskBarDisplay = <T,>({
174174
onMouseLeave={handleOnMouseLeave}
175175
onClick={task.onClick}
176176
>
177-
{hasChildren && (
177+
{hasOverlays && (
178178
<IconButton
179179
onClick={(e) => {
180180
e.stopPropagation();
@@ -193,8 +193,8 @@ const GanttTaskBarDisplay = <T,>({
193193
{task.name}
194194
</Typography>
195195
</div>
196-
{hasChildren &&
197-
task.children.map((childTask) => {
196+
{hasOverlays &&
197+
task.overlays.map((childTask) => {
198198
return (
199199
<div
200200
style={ganttTaskBarChildOverlayStyles(childTask)}

src/frontend/src/utils/gantt.utils.tsx

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -87,6 +87,7 @@ interface GanttTaskData<T> {
8787
blocking: GanttTaskData<T>[];
8888
children: GanttTaskData<T>[];
8989
events: GanttEvent[];
90+
overlays: GanttTaskData<T>[];
9091

9192
// Optional Values
9293
styles?: GanttTaskStyles;
@@ -388,6 +389,7 @@ export const transformWorkPackageToGanttTask = <T extends WorkPackage>(
388389
events: workPackage.designReviews.map(transformDesignReviewToGanttEvent),
389390
blocking: getBlockingGanttTasks(workPackage, allWorkPackages, transformWorkPackageToGanttTask),
390391
children: [],
392+
overlays: [],
391393

392394
tooltip: {
393395
upperRightDisplay: <UserDisplay user={workPackage.lead} label="Lead" />,
@@ -418,6 +420,7 @@ export const transformProjectToGanttTask = (project: ProjectPreview): GanttTask<
418420
children: project.workPackages
419421
.filter((workPackage) => workPackage.blockedBy.length === 0)
420422
.map((workPackage) => transformWorkPackageToGanttTask(workPackage, project.workPackages)),
423+
overlays: project.workPackages.map((wp) => transformWorkPackageToGanttTask(wp, project.workPackages)),
421424
events: [],
422425
tooltip: {
423426
upperRightDisplay: <UserDisplay user={project.lead} label="Lead" />,

0 commit comments

Comments
 (0)