Skip to content

Commit a707ff9

Browse files
author
Brian Genisio
committed
don't show a next button in the last question
1 parent 23ab24f commit a707ff9

File tree

3 files changed

+14
-17
lines changed

3 files changed

+14
-17
lines changed

data/answer.md

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -9,24 +9,24 @@ __Summary__
99
__Responses__
1010

1111
1. **Question 1**
12-
- Selected Answer: No answer selected
12+
- Selected Answer: D
1313
- Correct Answer: B, C
1414
- Result: ✗ Incorrect
1515

1616
2. **Question 2**
17-
- Selected Answer: B
17+
- Selected Answer: A
1818
- Correct Answer: A
19-
- Result: ✗ Incorrect
19+
- Result: ✓ Correct
2020

2121
3. **Question 3**
22-
- Selected Answer: C
22+
- Selected Answer: D
2323
- Correct Answer: B
2424
- Result: ✗ Incorrect
2525

2626
4. **Question 4**
27-
- Selected Answer: B
28-
- Correct Answer: B
29-
- Result: ✓ Correct
27+
- Selected Answer: B, D
28+
- Correct Answer: B, C
29+
- Result: ✗ Incorrect
3030

3131
__Practice Question__
3232

@@ -92,5 +92,5 @@ __Suggested Answers__
9292
- D
9393
- B - Correct
9494
- A
95-
- C
95+
- C - Correct
9696

data/question.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -65,5 +65,5 @@ __Suggested Answers__
6565

6666
- A
6767
- B - Correct
68-
- C
68+
- C - Correct
6969
- D

public/modules/mcq.js

Lines changed: 5 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -149,17 +149,16 @@ export function initMcq({ activity, state, postResults }) {
149149

150150
questionEl.appendChild(optionsEl);
151151

152-
// Add "Next" button for multi-select questions
153-
if (question.isMultiSelect) {
152+
// Add "Next" button for multi-select questions (not on last question)
153+
if (question.isMultiSelect && qIdx < mcq.questions.length - 1) {
154154
const nextButtonContainer = document.createElement('div');
155155
nextButtonContainer.className = 'mcq-next-button-container';
156156

157157
const nextButton = document.createElement('button');
158158
nextButton.className = 'button button-primary mcq-next-button';
159159
nextButton.textContent = 'Next';
160160
nextButton.type = 'button';
161-
const isLastQuestion = qIdx === mcq.questions.length - 1;
162-
// Always visible, but disabled if no answer selected or if it's the last question
161+
// Always visible, but disabled if no answer selected
163162
nextButton.disabled = true; // Initially disabled until answer is selected
164163
nextButton.setAttribute('aria-label', `Go to next question`);
165164

@@ -293,10 +292,8 @@ export function initMcq({ activity, state, postResults }) {
293292
const nextButton = questionEl.querySelector('.mcq-next-button');
294293
if (nextButton) {
295294
const hasSelection = selectedAnswers[questionId].length > 0;
296-
const questionIndex = parseInt(questionEl.getAttribute('data-question-index'), 10);
297-
const isLastQuestion = questionIndex === mcq.questions.length - 1;
298-
// Disable if no answer selected or if it's the last question
299-
nextButton.disabled = !hasSelection || isLastQuestion;
295+
// Disable if no answer selected
296+
nextButton.disabled = !hasSelection;
300297
}
301298
}
302299
} else {

0 commit comments

Comments
 (0)