Refresh student_quiz_report_v2 UI#76
Merged
Merged
Conversation
Redesigns the v2 report template around a mobile-first card layout, with responsive breakpoints at md (768px) and lg (1024px) so it also reads well on desktop. No backend or data-contract changes — the router still passes the same `report` context. Key changes: - Container: centered card that grows from max-w-md to max-w-5xl. - Header: solid blue bar; student/test fields stack on mobile, single row on md+. - Overall Score: standalone card (always above Recommendation). - Overall Performance Summary: 2-col on mobile, 6-col on md+ (6 metric tiles: Total Marks, Correct, Percentage, Skipped, Wrong, Accuracy). - Subject-wise Performance: stacked on mobile, 3-col on md+; each card has a colored left border and a 4-cell pair-table (Marks/Skipped, Wrong/Correct, Percentage/Accuracy). - Chapter-wise Performance: tabbed single-table view (Physics/Chemistry/ Mathematics or Biology for NEET) with icon headers for Correct/Wrong/Skipped. - Removed the in-page Print/Save button. - Print template (student_quiz_report_v2_print.html) left untouched. Also adds `ui/` to .gitignore (local Next.js mockup scratch dir). Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Zero-attempt v2 records store `accuracy` (overall and per-subject) as null, since correct / (correct + wrong) is 0/0. The new template formats every summary and subject-card number, so null reaches `%.2f` and raises TypeError during render. Fall back to 0 for marks_scored, num_correct/wrong/skipped, percentage and accuracy so a "student skipped everything" session renders as 0/0.00% instead of crashing. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
suryabulusu
approved these changes
May 5, 2026
suryabulusu
left a comment
suryabulusu
left a comment
Collaborator
There was a problem hiding this comment.
looks good with some comments on discord
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
2 participants
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
student_quiz_report_v2.htmltemplate around a mobile-first, card-based layout with responsive breakpoints atmd(768px) andlg(1024px), so it reads well on both phones and desktop.student_quiz_report_v2_print.htmlis untouched, so PDF output is unaffected.ui/to.gitignore(local Next.js mockup scratch dir used during design).Follow-up updates by @suryabulusu
maininto this branch.100 / 180.Review Quizsupport for v2 reports when a launch-token-backed quiz review link can be generated.Test plan
marks_scored / max_marks_possible.Review Quizbutton when a quiz id is available.?format=pdfstill renders using the untouched print template.🤖 Generated with Claude Code