diff --git a/src/assets/wise5/classroomMonitor/classroomMonitorComponents/nodeGrading/filter-components/filter-components.component.spec.ts b/src/assets/wise5/classroomMonitor/classroomMonitorComponents/nodeGrading/filter-components/filter-components.component.spec.ts index f2b9c28cf47..8c3541f8b88 100644 --- a/src/assets/wise5/classroomMonitor/classroomMonitorComponents/nodeGrading/filter-components/filter-components.component.spec.ts +++ b/src/assets/wise5/classroomMonitor/classroomMonitorComponents/nodeGrading/filter-components/filter-components.component.spec.ts @@ -35,6 +35,8 @@ function onlyOneComponent() { type: 'MultipleChoice' } as ComponentContent ]; + component.selectedComponents = component.components; + component.ngOnChanges(); fixture.detectChanges(); select = await loader.getHarness(MatSelectHarness); }); @@ -57,6 +59,7 @@ function moreThanOneComponent() { type: 'OpenResponse' } as ComponentContent ]; + component.selectedComponents = [component.components[0]]; component.ngOnChanges(); fixture.detectChanges(); select = await loader.getHarness(MatSelectHarness); @@ -66,14 +69,18 @@ function moreThanOneComponent() { const options = await select.getOptions(); expect(options.length).toBe(2); expect(await options[0].isSelected()).toBe(true); - expect(await options[1].isSelected()).toBe(true); + expect(await options[1].isSelected()).toBe(false); }); it('clicking on an option should emit selected components', async () => { const spy = spyOn(component.componentsChange, 'emit').and.callThrough(); await select.open(); const options = await select.getOptions(); - await options[0].click(); + await options[1].click(); expect(spy).toHaveBeenCalledWith([ + { + id: 'c1', + type: 'MultipleChoice' + } as ComponentContent, { id: 'c2', type: 'OpenResponse' diff --git a/src/assets/wise5/classroomMonitor/classroomMonitorComponents/nodeGrading/filter-components/filter-components.component.ts b/src/assets/wise5/classroomMonitor/classroomMonitorComponents/nodeGrading/filter-components/filter-components.component.ts index 58872239791..b42dfe2df5f 100644 --- a/src/assets/wise5/classroomMonitor/classroomMonitorComponents/nodeGrading/filter-components/filter-components.component.ts +++ b/src/assets/wise5/classroomMonitor/classroomMonitorComponents/nodeGrading/filter-components/filter-components.component.ts @@ -19,13 +19,12 @@ export class FilterComponentsComponent { @Output() componentsChange: EventEmitter = new EventEmitter< ComponentContent[] >(); - protected selectedComponents: ComponentContent[]; + @Input() selectedComponents: ComponentContent[]; protected selectedText: string; constructor(private componentTypeService: ComponentTypeService) {} ngOnChanges(): void { - this.selectedComponents = this.components; this.updateSelectedText(); } diff --git a/src/assets/wise5/classroomMonitor/classroomMonitorComponents/nodeGrading/node-grading/node-grading.component.html b/src/assets/wise5/classroomMonitor/classroomMonitorComponents/nodeGrading/node-grading/node-grading.component.html index a6297d513c9..6a9f3a65c0d 100644 --- a/src/assets/wise5/classroomMonitor/classroomMonitorComponents/nodeGrading/node-grading/node-grading.component.html +++ b/src/assets/wise5/classroomMonitor/classroomMonitorComponents/nodeGrading/node-grading/node-grading.component.html @@ -37,7 +37,7 @@

Question Summaries

@@ -56,6 +56,7 @@

Question Summaries

Class Responses

diff --git a/src/assets/wise5/classroomMonitor/classroomMonitorComponents/nodeGrading/node-grading/node-grading.component.ts b/src/assets/wise5/classroomMonitor/classroomMonitorComponents/nodeGrading/node-grading/node-grading.component.ts index beb33082d38..f55136530f7 100644 --- a/src/assets/wise5/classroomMonitor/classroomMonitorComponents/nodeGrading/node-grading/node-grading.component.ts +++ b/src/assets/wise5/classroomMonitor/classroomMonitorComponents/nodeGrading/node-grading/node-grading.component.ts @@ -103,7 +103,7 @@ export class NodeGradingComponent implements OnInit, OnDestroy, OnChanges { component['displayIndex'] = index + 1; return component; }); - this.visibleComponents = this.components; + this.visibleComponents = [this.components[0]]; this.numRubrics = this.node.getNumRubrics(); this.setPeriod(); } @@ -144,4 +144,9 @@ export class NodeGradingComponent implements OnInit, OnDestroy, OnChanges { event.preventDefault(); this.summariesVisible = !this.summariesVisible; } + + protected selectSummary(componentIndex: number): void { + this.selectedComponent.setValue(componentIndex); + this.visibleComponents = [this.components[componentIndex]]; + } } diff --git a/src/messages.xlf b/src/messages.xlf index 80d53987af0..19bb956924d 100644 --- a/src/messages.xlf +++ b/src/messages.xlf @@ -14629,7 +14629,7 @@ The branches will be removed but the steps will remain in the unit. Showing / questions src/assets/wise5/classroomMonitor/classroomMonitorComponents/nodeGrading/filter-components/filter-components.component.ts - 33 + 32