Skip to content

Conversation

@GumisC4
Copy link
Contributor

@GumisC4 GumisC4 commented Dec 14, 2025

No description provided.

Copilot AI review requested due to automatic review settings December 14, 2025 16:34
@GumisC4 GumisC4 self-assigned this Dec 14, 2025
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR implements per-question advanced mode functionality, replacing the global advanced mode toggle with individual advanced toggles for each question. The global "advanced mode" switch now serves as the default setting for newly created questions. Each question can independently enable/disable advanced features (images, explanations, and answer images) through its own toggle.

Key Changes

  • Refactored advanced mode from global quiz-level to per-question level with an advanced boolean property
  • Modified sanitizeQuestions to use per-question advanced flags instead of a global parameter
  • Added per-question advanced mode toggle switch in the question form UI

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 6 comments.

File Description
src/components/quiz/quiz-editor.tsx Extends Question type with optional advanced property; refactors sanitizeQuestions to use per-question flags; updates initialization logic to detect and set advanced mode per question; updates global advanced mode label to clarify it's the default for new questions
src/components/quiz/question-form.tsx Adds per-question advanced toggle switch; replaces advancedMode prop with question.advanced property; duplicates multiple-choice checkbox in non-advanced view for better UX

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@Antoni-Czaplicki Antoni-Czaplicki linked an issue Dec 14, 2025 that may be closed by this pull request
@Antoni-Czaplicki Antoni-Czaplicki force-pushed the feat/107-add-advanced-mode-per-question branch from 24683fd to 0cf189c Compare December 14, 2025 20:06
Copy link
Member

@Antoni-Czaplicki Antoni-Czaplicki left a comment

Choose a reason for hiding this comment

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

Przetestuj proszę kod zanim go wrzucisz

Image

Copilot AI review requested due to automatic review settings January 4, 2026 19:44
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 6 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Member

@Antoni-Czaplicki Antoni-Czaplicki left a comment

Choose a reason for hiding this comment

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

Lgtm

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@Antoni-Czaplicki Antoni-Czaplicki merged commit ff3e177 into main Jan 4, 2026
10 checks passed
@Antoni-Czaplicki Antoni-Czaplicki deleted the feat/107-add-advanced-mode-per-question branch January 4, 2026 20:01
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.

Włączanie trybu zaawansowanego dla każdego pytania osobno

3 participants