Feat: Add quiz type pin image#2363
Merged
Merged
Conversation
- Introduced a new question type 'pin_image' to the quiz system, allowing students to drop a pin on an image as an answer. - Updated the quiz handling logic to support validation and processing of pin coordinates. - Enhanced the frontend components to include the new pin_image type, ensuring proper rendering and interaction. - Improved backend handling for pin_image answers, including file path management and answer validation. - Updated relevant views to display pin submissions and instructor references correctly.
…e image handling - Removed unnecessary field value assignment in PinImage component. - Enhanced FormPinImage to ensure proper handling of image uploads by checking for the presence of the option before accessing its properties. - Updated the TutorCore API reference for drawing on images to ensure compatibility with the latest changes.
- Added a TODO comment indicating that the icon for the 'Pin on Image' quiz type is not final, to ensure future updates address this aspect.
…n image handling - Removed the add_pin_image_quiz_file_paths_for_deletion method and its associated documentation as it is no longer needed. - Simplified the process_pin_image_question_answer method by removing unnecessary parameters to streamline its functionality. - Updated comments for clarity regarding the handling of pin image questions.
- Updated variable names from 'w' and 'h' to 'width' and 'height' for better readability and understanding of their purpose in the FormPinImage component. - Ensured consistent naming conventions to enhance code maintainability.
harunollyo
requested changes
Feb 24, 2026
harunollyo
requested changes
Mar 3, 2026
…lity and improved reference mask handling
…ctions and enhance UI elements for pin image questions
…iz-type-pin-image
- Introduced a new variable to handle 'pin_image' question type in the review answers template. - This enhancement allows for better handling of different question types during quiz attempts.
- Updated the QuestionList component to filter out both 'Draw on Image' and 'Pin on Image' question types when legacy learning mode is enabled. - Added exception handling in the QuizBuilder class to throw appropriate messages when attempting to use 'Pin on Image' questions in legacy mode.
…include new question type. This change enhances the quiz functionality by adding support for 'pin-image' questions while cleaning up unused templates.
… and 'pin_image' question types. This change simplifies the quiz functionality by focusing on supported question types and enhancing code maintainability.
sazedul-haque
approved these changes
Apr 3, 2026
shewa12
reviewed
Apr 3, 2026
| $legacy_draw_image_message = __( 'Draw on Image questions are not available when Legacy learning mode is enabled.', 'tutor' ); | ||
| throw new \Exception( $legacy_draw_image_message ); | ||
| } | ||
| if ( 'pin_image' === $question_type && tutor_utils()->is_legacy_learning_mode() ) { |
Collaborator
There was a problem hiding this comment.
Use the constant for Quiz Type
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
4 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.
No description provided.