Merge pull request #3940 from NoelDeMartin/MOBILE-4350
MOBILE-4350: Fix sequential navigation summarymain
commit
55b1b527c0
|
@ -104,7 +104,7 @@
|
|||
<!-- List of questions of the summary table. -->
|
||||
<ng-container *ngFor="let question of summaryQuestions">
|
||||
<ion-item *ngIf="question.type !== 'description' && question.questionnumber"
|
||||
(click)="changePage(question.page, false, question.slot)"
|
||||
(click)="!isSequential && canReturn && changePage(question.page, false, question.slot)"
|
||||
[attr.aria-label]="'core.question.questionno' | translate:{$a: question.questionnumber}"
|
||||
[detail]="!isSequential && canReturn" [button]="!isSequential && canReturn">
|
||||
<ion-label>
|
||||
|
|
|
@ -412,7 +412,7 @@ export class AddonModQuizPlayerPage implements OnInit, OnDestroy, CanLeave {
|
|||
.filter(question => AddonModQuiz.isQuestionUnanswered(question))
|
||||
.length;
|
||||
|
||||
if (unansweredCount > 0) {
|
||||
if (!this.isSequential && unansweredCount > 0) {
|
||||
const warning = Translate.instant(
|
||||
'addon.mod_quiz.submission_confirmation_unanswered',
|
||||
{ $a: unansweredCount },
|
||||
|
|
|
@ -132,8 +132,14 @@ Feature: Navigate through a quiz in the app
|
|||
And I should find "Not yet answered" within "2" "ion-item" in the app
|
||||
And I should find "Not yet answered" within "3" "ion-item" in the app
|
||||
|
||||
When I press "Not yet answered" within "3" "ion-item" in the app
|
||||
Then I should not find "Text of the third question" in the app
|
||||
|
||||
When I press "Submit all and finish" in the app
|
||||
And I press "Submit" near "Once you submit" in the app
|
||||
Then I should find "Once you submit" in the app
|
||||
But I should not find "Questions without a response" in the app
|
||||
|
||||
When I press "Submit" near "Once you submit" in the app
|
||||
Then I should find "Review" in the app
|
||||
And I should find "Text of the first question" in the app
|
||||
And I should find "Text of the second question" in the app
|
||||
|
|
Loading…
Reference in New Issue