Skip to content

feat(recorder): add manual markers global shortcut and confirmation t… - #39

Open
aaminashihab wants to merge 1 commit into
microsoft:mainfrom
aaminashihab:feat/manual-markers
Open

feat(recorder): add manual markers global shortcut and confirmation t…#39
aaminashihab wants to merge 1 commit into
microsoft:mainfrom
aaminashihab:feat/manual-markers

Conversation

@aaminashihab

@aaminashihab aaminashihab commented Aug 4, 2026

Copy link
Copy Markdown

Summary of Changes

  • Revived manual markers via a global shortcut (CommandOrControl+Shift+M) without interrupting recordings or showing blocking dialogs.
  • Added renderer bridge event onMarkerAdded in electron/preload.cjs and common/ipc.ts.
  • Added a non-blocking confirmation toast in src/RecordingControls.tsx with fade-out keyframes in src/App.css.
  • Updated docs/future-features.md to reflect the live status of the feature.
  • Added unit test in electron/recorder/controller.test.ts verifying marker persistence during active recordings.

Validation

  • npm run typecheck - 0 errors
  • npm run typecheck:evals - 0 errors
  • npm test - 76 / 76 tests passing
  • npm run build - Build succeeded cleanly
    @microsoft-github-policy-service agree

@aaminashihab

Copy link
Copy Markdown
Author

@microsoft-github-policy-service agree

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