Skip to content

Improve async PI student user experience: step banner, button fix, and prompt update#1178

Merged
bnmnetp merged 8 commits intoRunestoneInteractive:mainfrom
sethbern:fix-next-question-placement
Mar 24, 2026
Merged

Improve async PI student user experience: step banner, button fix, and prompt update#1178
bnmnetp merged 8 commits intoRunestoneInteractive:mainfrom
sethbern:fix-next-question-placement

Conversation

@sethbern
Copy link
Copy Markdown
Contributor

This PR fixes some usability issues observed during testing of async peer instruction.

  • There is now a step progress banner. Now see a colored banner (Step 1 of 3, 2 of 3, 3 of 3) so they always know where they are in the PI flow. This was confusing before since there is no live instructor
  • Previously the "Next Question" button was being pushed far to the right side of the screen, now it stays on the screen regardless of screen size
    • It now says "Done" when on the final question
  • Updated prompt to focus on asking students to justify their reasoning instead of adversarially pushing wrong answers
image image image

@sethbern sethbern requested a review from bnmnetp as a code owner March 16, 2026 20:50
@bnmnetp
Copy link
Copy Markdown
Member

bnmnetp commented Mar 16, 2026

OK, I gotta ask... Have you verified that regular PI continues to function normally with these changes in place?

@sethbern
Copy link
Copy Markdown
Contributor Author

Yep, these changes don't touch any sync PI code paths or grading logic, they're scoped to the async template, the async controller function, and the LLM prompt.

I tested sync PI and both text chat and in-person group discussions, and they work as they should.

@sethbern sethbern marked this pull request as draft March 18, 2026 15:03
@sethbern sethbern marked this pull request as ready for review March 18, 2026 18:01
@bnmnetp bnmnetp merged commit ea2187a into RunestoneInteractive:main Mar 24, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants