Skip to content

Add responsive multi-mode quiz practice interface#4

Open
apple246680 wants to merge 1 commit intomainfrom
codex/design-study-question-system-40yheu
Open

Add responsive multi-mode quiz practice interface#4
apple246680 wants to merge 1 commit intomainfrom
codex/design-study-question-system-40yheu

Conversation

@apple246680
Copy link
Owner

Summary

  • create a responsive single-page quiz interface with light and dark themes
  • implement free practice, custom range, and mock exam flows with bookmarks, progress, and scoring
  • add progress tracking, jump navigation, and range selection support for the question bank

Testing

  • Not run (not applicable)

https://chatgpt.com/codex/tasks/task_e_6904a708f2788320922e0e5eecd66927

Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment on lines +202 to +206
function startExamMode() {
resetState();
state.mode = "exam";
state.currentQuestions = pickRandom(state.questions, 50);
activateQuiz();

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Badge Prevent starting quiz before question data loads

The mode handlers assume the question bank is already populated when the buttons are clicked. Because the buttons are wired up before loadQuestions() resolves, a user on a slow or failed fetch can start a quiz while state.questions is still empty, which renders a blank quiz and yields Infinity when finishExam divides by total (0 questions). Consider disabling the mode buttons until the fetch completes or guard these functions so they wait for the questions to load before copying them into state.currentQuestions.

Useful? React with 👍 / 👎.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant