Skip to content

Acknowledge 'sentence unclear' feedback instead of closing silently - #175

Merged
sidprasad merged 1 commit into
mainfrom
acknowledge-unclear-feedback
Jul 22, 2026
Merged

Acknowledge 'sentence unclear' feedback instead of closing silently#175
sidprasad merged 1 commit into
mainfrom
acknowledge-unclear-feedback

Conversation

@sidprasad

Copy link
Copy Markdown
Contributor

What changed

When a student reports an english-to-LTL sentence as confusing or unclear, the modal previously just closed on submit — no confirmation, no guidance. Students were left wondering whether the feedback went anywhere and whether they still had to answer.

  • exercise.html: a persistent note now appears under the "Is this English sentence confusing or unclear?" link after a successful submission: "Thank you for your feedback. Please answer with your best-guess interpretation of the sentence." It stays visible while the student answers, and carries role="status" so screen readers announce it when it appears (message is conveyed in text, no color-only signaling).
  • ltltoengrater.html: on success, the modal closes and the note is revealed. A failed submission now shows an alert asking the student to retry — previously it only wrote to the console, making failure indistinguishable from success.
  • Version bumped to 2.1.3; CHANGELOG entry added.

Why

Student-reported UX gap: flagging a sentence as unclear gave no acknowledgment and no instruction to proceed, so the report felt like a dead end.

Notes for review

  • The question flow is intentionally unchanged: students still answer, and responses on flagged questions are logged normally. Excluding or down-weighting flagged answers in the misconception model would be a separate change to the answer-logging path.
  • The rater modal is only included from exercise.html; the JS guards on the ack element existing, so the modal remains safe to reuse elsewhere.

Submitting the "Is this English sentence confusing or unclear?" report
previously just hid the modal: no confirmation, and no guidance on what
to do next. Students were left wondering whether they still had to
answer. A persistent note now appears under the question link telling
them the feedback was recorded and to answer with their best-guess
interpretation (role="status" so screen readers announce it). A failed
submission now alerts the student instead of logging only to the
console, which was indistinguishable from success.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@sidprasad
sidprasad merged commit f0b6873 into main Jul 22, 2026
2 checks passed
@sidprasad
sidprasad deleted the acknowledge-unclear-feedback branch July 22, 2026 11:10
sidprasad added a commit that referenced this pull request Jul 22, 2026
Main's #175 shipped as 2.1.3, so this PR's changelog entries and
version.html move to 2.1.4.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
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.

1 participant