Skip to content

AI Tutor: refactor Reflection components into subdirectory with updated styles#72547

Merged
Erin007 merged 2 commits intostagingfrom
erin-reflection-refactor
May 7, 2026
Merged

AI Tutor: refactor Reflection components into subdirectory with updated styles#72547
Erin007 merged 2 commits intostagingfrom
erin-reflection-refactor

Conversation

@Erin007
Copy link
Copy Markdown
Contributor

@Erin007 Erin007 commented May 6, 2026

Reducing the diff for #72520

Summary

  • Moves ReflectionBox, LessonObjectiveReflection, LessonReflection, and reflection.module.scss from the lessonDeepDive/ root into a Reflection/ subdirectory for better organization
  • Replaces MUI Typography/Button with native HTML elements styled via a new comprehensive SCSS module
  • Updates rating button labels to Struggling / Getting there / Got it (matching Figma), and submit button text to "Start practicing"
  • Adds unitLabel and onNext props to ReflectionBox; unitLabel is now surfaced from the lessons controller
  • Updates test selectors to match the new button labels and textarea structure

Extracted from #72520 to make review easier.

Test plan

  • All 6 ReflectionBoxTest assertions pass (yarn test:unit test/unit/aiTutor/views/lessonDeepDive/ReflectionBoxTest.tsx)
  • No new TypeScript errors in changed files (yarn run typecheck)
  • Lint clean (./tools/hooks/pre-commit)
  • Verify reflection slide renders with unit label and navigates to next slide on submit

🤖 Generated with Claude Code

…ed styles

Move ReflectionBox, LessonObjectiveReflection, LessonReflection, and
reflection.module.scss into a Reflection/ subdirectory. Replace MUI
Typography/Button with native elements styled via SCSS. Update rating
button labels (Struggling/Getting there/Got it) and submit text. Add
unitLabel and onNext props. Expose unitLabel from the lessons controller.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@Erin007 Erin007 merged commit 1da9223 into staging May 7, 2026
8 checks passed
@Erin007 Erin007 deleted the erin-reflection-refactor branch May 7, 2026 11:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants