Skip to content

feat(review-tutor): show a pulsing ember dot while the tutor answers - #69

Merged
ElbertePlinio merged 1 commit into
mainfrom
feat/review-tutor-ask-loader
Aug 25, 2026
Merged

feat(review-tutor): show a pulsing ember dot while the tutor answers#69
ElbertePlinio merged 1 commit into
mainfrom
feat/review-tutor-ask-loader

Conversation

@ElbertePlinio

@ElbertePlinio ElbertePlinio commented Aug 24, 2026

Copy link
Copy Markdown
Member

Refs #64 (human smoke feedback on #68). Stacked on #67; part of the batched Review Tutor chain.

What changed

  • The busy Ask button shows one cue: a pulsing ember dot beside Sending → Queued → Answering, replacing the bare "Running" text. Nothing else changes; the answer block still appears only once content exists.
  • Reduced motion: the dot renders solid. The dot is aria-hidden; announcements and the button's accessible name are unchanged.

Tested

  • Page tests for the label sequence and dot presence only while busy; bun run check green.
  • Visual acceptance in Elberte's live smoke (the first version with an extra progress line and early Answer block was rejected as overdesign and removed).

Human gate: merge only on Elberte's authorization.

@ElbertePlinio
ElbertePlinio force-pushed the feat/review-tutor-structure-core branch from 22f5904 to 39adf78 Compare August 24, 2026 21:20
@ElbertePlinio
ElbertePlinio force-pushed the feat/review-tutor-ask-loader branch from 1079c29 to 1800cce Compare August 24, 2026 21:20
@ElbertePlinio ElbertePlinio changed the title feat(review-tutor): show an ember loading state while the tutor answers feat(review-tutor): show a pulsing ember dot while the tutor answers Aug 24, 2026
@ElbertePlinio
ElbertePlinio force-pushed the feat/review-tutor-structure-core branch from 39adf78 to b2dfb4d Compare August 24, 2026 21:30
@ElbertePlinio
ElbertePlinio force-pushed the feat/review-tutor-ask-loader branch from 1800cce to 447bfdd Compare August 24, 2026 21:30
@ElbertePlinio
ElbertePlinio force-pushed the feat/review-tutor-structure-core branch from b2dfb4d to 497c5fa Compare August 24, 2026 21:30
@ElbertePlinio
ElbertePlinio force-pushed the feat/review-tutor-ask-loader branch from 447bfdd to 90e73e5 Compare August 24, 2026 21:30
@ElbertePlinio
ElbertePlinio changed the base branch from feat/review-tutor-structure-core to main August 25, 2026 19:09
Replace the plain Running label with a pulsing ember dot and Sending,
Queued, and Answering labels on the busy Ask button.
@ElbertePlinio
ElbertePlinio force-pushed the feat/review-tutor-ask-loader branch from 90e73e5 to cc948f1 Compare August 25, 2026 19:12
@ElbertePlinio
ElbertePlinio merged commit 51c29db into main Aug 25, 2026
4 checks 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.

1 participant