Tyler/StepForge
1
0
Fork 0
Code Issues 1 Pull Requests 1 Actions Packages Projects Releases Wiki Activity

Issue #4: Show recording directions after capture starts #11

Closed
Tyler wants to merge 0 commits from tyler/fix/issue-4-recording-notification AGit into main
pull from: tyler/fix/issue-4-recording-notification
merge into: :main
Conversation 0 Commits - Files Changed -
Tyler commented 2026-06-12 17:23:59 +00:00
Owner
Copy Link
Copy Source

Improvement Type

  • Improvement
  • Bug
  • Documentation
  • Tests
  • Git / workflow
  • Other

Issue Type

  • Bug
  • Improvement
  • Task
  • Documentation

Summary

Moves the recording guidance into the recording-start flow so it appears immediately in-app instead of as a generic Windows warning before the user is ready.

Current Behavior

The recording notice can appear before recording is actually started, and the control flow exposes extra actions that make the start/pause experience harder to follow.

Expected Behavior

Show a friendly in-app acknowledgment as soon as recording starts, require the user to confirm it, and keep the recording controls focused on a single start action.

Steps To Reproduce

  1. Open the library view.
  2. Start a new guide or open an existing one.
  3. Click record.
  4. Observe when the guidance appears and how the recording controls behave.

Proposed Changes or Acceptance Criteria

  • Show the message from the recording-start path instead of on editor open.
  • Use an in-app UI element that blends with the app instead of the default Windows warning popup.
  • Require the user to acknowledge the message before continuing.
  • Keep the flow immediate when recording begins.

Testing Notes

  • Ran npm test.
  • Ran bash tests/run_test.sh.

Screenshots, Logs, or Extra Context

  • N/A

Deployment / Rollout Notes

  • No special rollout steps
  • Documentation only
  • Other

Notes

None.

Checklist

  • Scope is limited to the selected area above.
  • Related docs or comments were updated if needed.
  • I ran bash tests/run_test.sh.
  • The PR references a issue number.
  • Any follow-up work is tracked in TODO.md or an issue.
## Improvement Type - [x] Improvement - [ ] Bug - [ ] Documentation - [ ] Tests - [ ] Git / workflow - [ ] Other ## Issue Type - [ ] Bug - [x] Improvement - [ ] Task - [ ] Documentation ## Summary Moves the recording guidance into the recording-start flow so it appears immediately in-app instead of as a generic Windows warning before the user is ready. ## Current Behavior The recording notice can appear before recording is actually started, and the control flow exposes extra actions that make the start/pause experience harder to follow. ## Expected Behavior Show a friendly in-app acknowledgment as soon as recording starts, require the user to confirm it, and keep the recording controls focused on a single start action. ## Steps To Reproduce 1. Open the library view. 2. Start a new guide or open an existing one. 3. Click record. 4. Observe when the guidance appears and how the recording controls behave. ## Proposed Changes or Acceptance Criteria - Show the message from the recording-start path instead of on editor open. - Use an in-app UI element that blends with the app instead of the default Windows warning popup. - Require the user to acknowledge the message before continuing. - Keep the flow immediate when recording begins. ## Testing Notes - Ran npm test. - Ran bash tests/run_test.sh. ## Screenshots, Logs, or Extra Context - N/A ## Deployment / Rollout Notes - [x] No special rollout steps - [ ] Documentation only - [ ] Other ### Notes None. ## Checklist - [x] Scope is limited to the selected area above. - [ ] Related docs or comments were updated if needed. - [x] I ran bash tests/run_test.sh. - [x] The PR references a issue number. - [ ] Any follow-up work is tracked in TODO.md or an issue.
Tyler added 1 commit 2026-06-12 17:23:59 +00:00
Fix guide editor issues 4-10
Template tests / tests (pull_request) Has been cancelled
Details
Template tests / tests (push) Has been cancelled
Details
f88ff0259e
Tyler added 3 commits 2026-06-13 19:10:33 +00:00
Require recording acknowledgment before hide
Template tests / tests (push) Has been cancelled
Details
e8f6e4cd09
Use in-app modal for recording start confirmation
Template tests / tests (push) Successful in 2m17s
Details
85b1f6f143
Prompt to start recording for library guides
Template tests / tests (push) Failing after 12m49s
Details
Template tests / tests (pull_request) Has been cancelled
Details
e46f01885b
Tyler added 1 commit 2026-06-13 19:25:50 +00:00
Remove new-capture follow-up toast
Template tests / tests (pull_request) Has been cancelled
Details
1a0fda71f5
Tyler added 1 commit 2026-06-13 19:30:59 +00:00
Sync tests workflow with main
Template tests / tests (pull_request) Successful in 1m48s
Details
dfd0139dc5
Tyler closed this pull request 2026-06-13 20:38:23 +00:00

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
No items
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
Tyler (Tyler Westbrook)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Tyler/StepForge#11