Redesign the pathway builder: warm, cohesive visual identity + better streaming flow - #61
Merged
Conversation
…only at the waypoint dot and pill
|
Deployment failed for project interactive-learning-experiences with the following error: Learn More: https://vercel.com/variantly?upgradeToPro=build-rate-limit |
Resolves conflicts in AssignToStudents.tsx by combining main's new functionality (already-assigned tracking, parentSessionId threading, richer completion messaging) with this branch's visual restyle.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
…sign) - useTelemetry.ts: move the two "always latest" ref assignments into an effect instead of the render body (react-hooks/refs) - PathwayWalkthrough.tsx: reset widgetDone by adjusting state during render (same idiom ActivityTrail already uses) instead of an effect (react-hooks/set-state-in-effect) - mcp/shell.tsx: pass the stepIndex WidgetTelemetryProvider now requires (0 is correct here, not a placeholder — this shell only ever renders one standalone widget, never a multi-step pathway) Verified all three were already broken on origin/main before this branch touched anything, by checking it out in a scratch worktree and running the same lint/build commands.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
A full visual and structural redesign of the teacher-facing pathway builder (
/), bringing it up to the same warm, playful identity/learnalready established for students — while keeping the calm, professional register a teacher's workspace needs.+ Add detaildisclosure instead of always being visible./learn's violet/pink/amber palette, spring motion (motion/react), and pressed-button physicality for primary CTAs — as an ambient wash rather than full-bleed saturation, since this page is read for minutes, not tapped through once. Added Lexend as a second, purpose-grounded display typeface (research-backed for reading proficiency), scoped locally to this page only.anchor-derived sections (prior knowledge, also touches) never appear out of backward-design order. The progress indicator is a segmented strip with live stage copy instead of an always-open technical log.widgetKindwhich is already known before the spec arrives.AssignToStudentsandLessonPlanUploadto match, without touching their logic.maintwice along the way (upload folded back into the builder, then lesson-plan excerpts shaping generation) and adapted both to the new design.Known follow-up
mainhas since added already-assigned-student tracking and aparentSessionIdprop toAssignToStudents.tsx(not in this branch yet) — that file will show a merge conflict against currentmainand needs one more fold-in pass before merging. Everything else here is clean againstmainas of this PR.Test plan
tsc --noEmit,eslint, andnext buildall cleanmain-side already-assigned-student flow (post-dates this branch, see Known follow-up)🤖 Generated with Claude Code