Skip to content

Commit 0369726

Browse files
feat(Teacher Summary): Remember anonymize responses options across discussion summaries (maximized windows and steps)
1 parent 63b4444 commit 0369726

3 files changed

Lines changed: 26 additions & 16 deletions

File tree

Lines changed: 17 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,17 @@
1+
import { InjectionToken } from '@angular/core';
2+
3+
export interface TeacherSummaryConfig {
4+
anonymizeStudentNames: boolean;
5+
}
6+
7+
export const TEACHER_SUMMARY_CONFIG = new InjectionToken<TeacherSummaryConfig>(
8+
'teacherSummaryConfig',
9+
{
10+
providedIn: 'root',
11+
factory: () => {
12+
return {
13+
anonymizeStudentNames: false
14+
};
15+
}
16+
}
17+
);
Lines changed: 6 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,12 @@
11
import { CommonModule } from '@angular/common';
2-
import { Component, EventEmitter, Input, OnInit, Output } from '@angular/core';
2+
import { Component, inject, Input, OnInit } from '@angular/core';
33
import { Component as WISEComponent } from '../../../common/Component';
44
import { TeacherSummaryDisplayComponent } from '../teacher-summary-display.component';
55
import { ComponentFactory } from '../../../common/ComponentFactory';
66
import { DiscussionTeacherComponent } from '../../../components/discussion/discussion-teacher/discussion-teacher.component';
77
import { MatSlideToggle } from '@angular/material/slide-toggle';
88
import { FormsModule } from '@angular/forms';
9+
import { TEACHER_SUMMARY_CONFIG } from '../TeacherSummaryConfig';
910

1011
@Component({
1112
imports: [CommonModule, DiscussionTeacherComponent, FormsModule, MatSlideToggle],
@@ -15,11 +16,7 @@ import { FormsModule } from '@angular/forms';
1516
<div [class.expanded]="expanded">
1617
<h2 class="mat-subtitle-1" i18n>Class Discussion</h2>
1718
<div class="mb-4 flex flex-wrap gap-4 justify-between items-center">
18-
<mat-slide-toggle
19-
[(ngModel)]="anonymizeResponses"
20-
(change)="anonymizeResponsesChanged()"
21-
i18n
22-
>
19+
<mat-slide-toggle [(ngModel)]="teacherSummaryConfig.anonymizeStudentNames" i18n>
2320
Hide student names
2421
</mat-slide-toggle>
2522
@if (component.content.anonymizeResponses) {
@@ -34,25 +31,21 @@ import { FormsModule } from '@angular/forms';
3431
[nodeId]="nodeId"
3532
[component]="component"
3633
[periodId]="periodId"
37-
[anonymizeResponses]="anonymizeResponses"
34+
[anonymizeResponses]="teacherSummaryConfig.anonymizeStudentNames"
3835
[mode]="'summary'"
3936
/>
4037
</div>
4138
`
4239
})
4340
export class DiscussionSummaryComponent extends TeacherSummaryDisplayComponent implements OnInit {
44-
@Input() anonymizeResponses: boolean;
4541
protected component: WISEComponent;
4642
@Input() expanded: boolean;
47-
@Output() anonymizeResponsesChange = new EventEmitter<boolean>();
43+
44+
protected teacherSummaryConfig = inject(TEACHER_SUMMARY_CONFIG);
4845

4946
ngOnInit(): void {
5047
let content = this.projectService.getComponent(this.nodeId, this.componentId);
5148
content = this.projectService.injectAssetPaths(content);
5249
this.component = new ComponentFactory().getComponent(content, this.nodeId);
5350
}
54-
55-
protected anonymizeResponsesChanged() {
56-
this.anonymizeResponsesChange.emit(this.anonymizeResponses);
57-
}
5851
}

src/messages.xlf

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -23237,21 +23237,21 @@ If this problem continues, let your teacher know and move on to the next activit
2323723237
<source>Class Discussion</source>
2323823238
<context-group purpose="location">
2323923239
<context context-type="sourcefile">src/assets/wise5/directives/teacher-summary-display/discussion-summary/discussion-summary.component.ts</context>
23240-
<context context-type="linenumber">16,17</context>
23240+
<context context-type="linenumber">17,18</context>
2324123241
</context-group>
2324223242
</trans-unit>
2324323243
<trans-unit id="6338860782944918625" datatype="html">
2324423244
<source> Hide student names </source>
2324523245
<context-group purpose="location">
2324623246
<context context-type="sourcefile">src/assets/wise5/directives/teacher-summary-display/discussion-summary/discussion-summary.component.ts</context>
23247-
<context context-type="linenumber">23,25</context>
23247+
<context context-type="linenumber">20,22</context>
2324823248
</context-group>
2324923249
</trans-unit>
2325023250
<trans-unit id="6793042667929328995" datatype="html">
2325123251
<source>Note: Students do not see each other&apos;s names in this activity.</source>
2325223252
<context-group purpose="location">
2325323253
<context context-type="sourcefile">src/assets/wise5/directives/teacher-summary-display/discussion-summary/discussion-summary.component.ts</context>
23254-
<context context-type="linenumber">27,32</context>
23254+
<context context-type="linenumber">24,29</context>
2325523255
</context-group>
2325623256
</trans-unit>
2325723257
<trans-unit id="7967732284579375900" datatype="html">

0 commit comments

Comments
 (0)