diff --git a/apps/web/src/components/practice/StartPracticeModal.tsx b/apps/web/src/components/practice/StartPracticeModal.tsx index 2948ec22..91a8415b 100644 --- a/apps/web/src/components/practice/StartPracticeModal.tsx +++ b/apps/web/src/components/practice/StartPracticeModal.tsx @@ -166,19 +166,6 @@ export function StartPracticeModal({ } }, [enabledParts]) - // Derive target skills from sessionMode (no duplicate BKT computation) - const targetSkillsInfo = useMemo(() => { - if (sessionMode.type === 'remediation') { - // In remediation mode, we have the weak skills to target - return { - targetedSkills: sessionMode.weakSkills, - hasData: true, - } - } - // In progression or maintenance mode, no specific targeting - return { targetedSkills: [], hasData: true } - }, [sessionMode]) - const generatePlan = useGenerateSessionPlan() const approvePlan = useApproveSessionPlan() const startPlan = useStartSessionPlan() @@ -719,75 +706,6 @@ export function StartPracticeModal({ ▼ - - {/* Target skills summary */} - {targetSkillsInfo.hasData && ( -
- {targetSkillsInfo.targetedSkills.length > 0 ? ( - <> - - Targeting: - - {targetSkillsInfo.targetedSkills.slice(0, 3).map((skill, i) => ( - - {skill.displayName} - {i < Math.min(targetSkillsInfo.targetedSkills.length, 3) - 1 && ','} - - ))} - {targetSkillsInfo.targetedSkills.length > 3 && ( - - +{targetSkillsInfo.targetedSkills.length - 3} more - - )} - - ) : ( - - Even distribution across all practicing skills - - )} -
- )} {/* Expanded config panel */} @@ -1214,98 +1132,6 @@ export function StartPracticeModal({ })} - - {/* Target skills info - in the grid for landscape layout */} - {targetSkillsInfo.hasData && ( -
0 - ? 'rgba(245, 158, 11, 0.08)' - : 'rgba(100, 116, 139, 0.08)' - : targetSkillsInfo.targetedSkills.length > 0 - ? 'rgba(245, 158, 11, 0.06)' - : 'rgba(100, 116, 139, 0.06)', - border: `1px solid ${ - isDark - ? targetSkillsInfo.targetedSkills.length > 0 - ? 'rgba(245, 158, 11, 0.2)' - : 'rgba(100, 116, 139, 0.2)' - : targetSkillsInfo.targetedSkills.length > 0 - ? 'rgba(245, 158, 11, 0.15)' - : 'rgba(100, 116, 139, 0.15)' - }`, - })} - > - {targetSkillsInfo.targetedSkills.length > 0 ? ( - <> -
- Focusing on weak skills: -
-
- {targetSkillsInfo.targetedSkills.map((skill) => ( - - {skill.displayName}{' '} - - ({Math.round(skill.pKnown * 100)}%) - - - ))} -
- - ) : ( -
- ✓ On track! Problems will be evenly distributed across all skills. -
- )} -
- )} {/* End settings-grid */}