feat(TeacherPresentation): Classroom Presentation view for Discussion student work by hirokiterashima · Pull Request #2351 · WISE-Community/WISE-Client

hirokiterashima · 2026-10-09T00:14:47Z

Notes

Please style as you see fit

Summary

Introduces the TeacherPresentation feature in the grading tool summary display. Teachers can select student work items (posts and comments) via checkboxes, customize student name display modes, edit the classroom prompt, and launch a dedicated presentation dialog designed to be projected on a classroom display.

Selection rules and store management are fully decoupled from discussion-specific hierarchies to support future component types. Authoring support is provided under content.teacher.presentation.prompt.

Changes

1. Presentation Dialog & Presentation Mode

TeacherPresentationDialogComponent: Fullscreen modal dialog styled for classroom projectors.
- Header displays unit and step titles, selected period name, student names display toggle (show, hide, anonymize), a refresh button, and an editable classroom prompt (with pencil icon).
- Renders the component in presentation mode (mode="presentation").
- Default prompt is sourced from content.teacher.presentation.prompt (with fallback to default localized prompt).
TeacherPresentationButtonComponent: "Present" button located on the summary display header.
- Enabled only when a specific single period is chosen and at least one item is selected.
- Shows tooltip explaining requirements when disabled.

2. State Management & Debounced Autosave

TeacherPresentationSelectionStore:
- Component-agnostic selection store managing selected student work IDs (selectedIds: Set<number>), studentNamesDisplay, and custom prompt.
- Generic selection API: isSelected(), select(), deselect(), toggle(), selectMultiple(), deselectMultiple(), setSelections(), and clearSelections().
- Debounced 1-second autosave to the backend (TeacherPresentationConfigService). Prompt changes persist immediately.

3. Discussion Component Integration & Hierarchical Selection

ClassResponseTeacherComponent:
- Implements discussion-specific selection logic:
  - Checking a reply automatically selects both the reply and its parent post.
  - Parent post checkbox is disabled/locked whenever any of its child replies are selected.
  - Unchecking a parent post deselects both the parent and all child replies.
  - Unchecking a reply deselects only that reply (parent remains selected).
- Checkboxes appear in summary display mode and are hidden in presentation view.
DiscussionTeacherComponent: Filters responses to display only teacher-selected posts and comments when in presentation mode.

4. Authoring Tool

In ComponentContent.ts, added teacher?: { presentation?: { prompt?: string } }.
In DiscussionAuthoringComponent, added authoring input for default presentation prompt under a "Classroom Presentation Settings" expansion panel.

5. Summary Display

DiscussionSummaryComponent:
- Extracted inline template to discussion-summary.component.html.
- Embeds teacher-presentation-button in the summary header.
- Binds TeacherPresentationSelectionStore across periods and component changes.

How to Test

Test with WISE-Community/WISE-API#362

Authoring Tool:
- Open or create a Discussion component in the Authoring Tool.
- Configure a default presentation prompt and reflection questions in the Presentation section.
Classroom Monitor / Teacher Summary:
- Navigate to the Discussion step summary in the Classroom Monitor.
- Verify the "Present Student Work" button is disabled until at least one post or comment is selected, showing a helper tooltip on hover.
- Select one or more posts and/or replies. Verify selecting a comment keeps the parent selected, and deselecting a parent unchecks comments.
- Enter answers into the Teacher Reflection expansion panel.
Classroom Presentation View:
- Click "Present Student Work" to open the modal.
- Toggle between Hide Names, Anonymize Names, and Show Names and verify author headers update accordingly.
- Edit and save a custom presentation prompt, or click Reset to default.
- Toggle presentation controls visibility via "Hide Controls" / "Show Controls".
- Refresh or reopen the dialog for the same period and ensure selected items and custom prompt persist.

…cussion summary

…teacher to choose a post or comment first

…o separate files

…nd clean up component code

…rom selection store

…component

qltysh · 2026-10-09T00:16:07Z

❌ 2 blocking issues (2 total)

Tool	Category	Rule	Count
qlty	Structure	Complex binary expression	2	❌

qltysh · 2026-10-09T00:16:09Z

+
+  private initPresentationStore(): void {
+    const runId = this.configService.getRunId();
+    if (runId && this.periodId != null && this.periodId !== -1 && this.nodeId && this.componentId) {


Complex binary expression [qlty:boolean-logic]

qltysh · 2026-10-09T00:16:09Z

+    this.componentType.set(componentType);
+    this.reflectionQuestions.set(reflectionQuestions);
+
+    if (runId && periodId && periodId !== -1 && nodeId && componentId) {


Complex binary expression [qlty:boolean-logic]

qltysh · 2026-10-09T00:17:16Z

Coverage Impact

Unable to calculate total coverage change because base branch coverage was not found.

Modified Files with Diff Coverage (8)

File	% Diff	Uncovered Line #s
.../discussion/discussion-teacher/discussion-teacher.component.ts	14.3%	18-57
...5/teacher-presentation/teacher-presentation-selection.store.ts	95.7%	53, 127-128, 157
.../teacher-presentation/teacher-presentation-dialog.component.ts	93.8%	45, 61
.../teacher-presentation/teacher-presentation-button.component.ts	78.4%	29-36, 47, 64
...cussion/discussion-authoring/discussion-authoring.component.ts	60.0%	27-28
...ion/class-response-teacher/class-response-teacher.component.ts	64.3%	84-87, 95, 108-114, 118
...e5/teacher-presentation/teacher-presentation-config.service.ts	58.3%	11-23
...ary-display/discussion-summary/discussion-summary.component.ts	15.4%	19-62
Total	75.3%

🤖 Increase coverage with AI coding...

In the `feat/teacher-presentation` branch, add test coverage for this new code:

- `src/assets/wise5/components/discussion/class-response-teacher/class-response-teacher.component.ts` -- Lines 84-87, 95, 108-114, and 118
- `src/assets/wise5/components/discussion/discussion-authoring/discussion-authoring.component.ts` -- Line 27-28
- `src/assets/wise5/components/discussion/discussion-teacher/discussion-teacher.component.ts` -- Line 18-57
- `src/assets/wise5/directives/teacher-summary-display/discussion-summary/discussion-summary.component.ts` -- Line 19-62
- `src/assets/wise5/teacher-presentation/teacher-presentation-button.component.ts` -- Lines 29-36, 47, and 64
- `src/assets/wise5/teacher-presentation/teacher-presentation-config.service.ts` -- Line 11-23
- `src/assets/wise5/teacher-presentation/teacher-presentation-dialog.component.ts` -- Lines 45 and 61
- `src/assets/wise5/teacher-presentation/teacher-presentation-selection.store.ts` -- Lines 53, 127-128, and 157

🚦 See full report on Qlty Cloud »

🛟 Help

Diff Coverage: Coverage for added or modified lines of code (excludes deleted files). Learn more.
Total Coverage: Coverage for the whole repository, calculated as the sum of all File Coverage. Learn more.
File Coverage: Covered Lines divided by Covered Lines plus Missed Lines. (Excludes non-executable lines including blank lines and comments.)
- Indirect Changes: Changes to File Coverage for files that were not modified in this PR. Learn more.

…ntation feature

…ation button text, and update translation file

hirokiterashima added 8 commits October 8, 2026 13:59

feat: implement TeacherPresentation classroom presentation feature

cec4b31

Remove student name anonymization toggle and expand template from dis…

732d26c

…cussion summary

feat: show tooltip on disabled Present Student Work button prompting …

db43c75

…teacher to choose a post or comment first

refactor: extract teacher presentation dialog template and styles int…

ba8a8f8

…o separate files

refactor: extract discussion-summary template to external html file a…

0cb0afa

…nd clean up component code

refactor(teacher-presentation): decouple discussion selection rules f…

338204d

…rom selection store

refactor: remove trailing newlines

e1807dd

refactor: update localization and UI labels for teacher presentation …

d856075

…component

hirokiterashima changed the title ~~feat(teacher-presentation): Classroom Presentation view for Discussion student work~~ feat(TeacherPresentation): Classroom Presentation view for Discussion student work Oct 9, 2026

qltysh Bot reviewed Oct 9, 2026

View reviewed changes

hirokiterashima mentioned this pull request Oct 9, 2026

feat(TeacherPresentation): Add API endpoints for config WISE-Community/WISE-API#362

Open

chore: sync package-lock.json with package.json

a4be366

hirokiterashima self-assigned this Oct 9, 2026

hirokiterashima added the enhancement New feature of any size or improvement (UI, performance, security) label Oct 9, 2026

hirokiterashima requested a review from breity October 9, 2026 00:39

hirokiterashima added 3 commits October 9, 2026 13:38

refactor(teacher-presentation): strip reflection questions from prese…

8a3a4ee

…ntation feature

refactor: update discussion authoring input handling, shorten present…

b0b54e9

…ation button text, and update translation file

update package-lock.json

87c0f84

Navigation Menu

Repository navigation

feat(TeacherPresentation): Classroom Presentation view for Discussion student work - #2351

feat(TeacherPresentation): Classroom Presentation view for Discussion student work#2351
hirokiterashima wants to merge 12 commits into
developfrom
feat/teacher-presentation

hirokiterashima commented Oct 9, 2026 •

edited

Loading

Uh oh!

qltysh Bot commented Oct 9, 2026 •

edited

Loading

Uh oh!

qltysh Bot Oct 9, 2026

Uh oh!

Uh oh!

qltysh Bot Oct 9, 2026

Uh oh!

qltysh Bot commented Oct 9, 2026 •

edited

Loading

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

Conversation

hirokiterashima commented Oct 9, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Notes

Summary

Changes

1. Presentation Dialog & Presentation Mode

2. State Management & Debounced Autosave

3. Discussion Component Integration & Hierarchical Selection

4. Authoring Tool

5. Summary Display

How to Test

Uh oh!

qltysh Bot commented Oct 9, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

❌ 2 blocking issues (2 total)

Uh oh!

qltysh Bot Oct 9, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

qltysh Bot Oct 9, 2026

Choose a reason for hiding this comment

Uh oh!

qltysh Bot commented Oct 9, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

hirokiterashima commented Oct 9, 2026 •

edited

Loading

qltysh Bot commented Oct 9, 2026 •

edited

Loading

qltysh Bot commented Oct 9, 2026 •

edited

Loading