Merge pull request #2924 from ever-co/2920-fix-actions-menu--remove-b…

…lank-space-after-adding-a-plan-to-tomorrow-plan fix: Action menu | remove blank space after adding a task to tomorrow plan
ever-co · Aug 17, 2024 · f1ba859 · f1ba859
2 parents 928720c + 148e8b4
commit f1ba859
Showing 1 changed file with 11 additions and 10 deletions.
diff --git a/apps/web/lib/features/task/task-card.tsx b/apps/web/lib/features/task/task-card.tsx
@@ -620,15 +620,16 @@ function TaskCardMenu({
 														/>
 													</li>
 												)}
-
-												<li className="mb-2">
-													<PlanTask
-														planMode="tomorow"
-														taskId={task.id}
-														employeeId={profile?.member?.employeeId ?? ''}
-														taskPlannedForTomorrow={taskPlannedTomorrow}
-													/>
-												</li>
+												{!taskPlannedTomorrow && (
+													<li className="mb-2">
+														<PlanTask
+															planMode="tomorow"
+															taskId={task.id}
+															employeeId={profile?.member?.employeeId ?? ''}
+															taskPlannedForTomorrow={taskPlannedTomorrow}
+														/>
+													</li>
+												)}
 												<li className="mb-2">
 													<PlanTask
 														planMode="custom"
@@ -774,7 +775,7 @@ export function PlanTask({
 				)}
 				{planMode === 'tomorow' && !taskPlannedForTomorrow && (
 					<span>
-						{isPending ? (
+						{isPending || createDailyPlanLoading ? (
 							<ReloadIcon className="animate-spin mr-2 h-4 w-4" />
 						) : (
 							t('dailyPlan.PLAN_FOR_TOMORROW')