fix: HUD blocked screen clicks during recording; shrink and free-drag the widget - #22
Merged
Merged
Conversation
…ding; shrink and free-drag the recording widget The webcam-bubble fix widened the HUD window to the full display during recording, but setHudOverlayMousePassthrough still unconditionally forced the whole window non-click-through whenever recording was active - so any recording with the webcam preview on silently blocked every click to the rest of the screen for the recording's duration. Generalizes the full-canvas mode to every recording on platforms with OS passthrough support (macOS, Windows 11+), reusing the existing hover-driven click-through/interactive toggling instead of forcing capture. This lets the whole recording-control bar - not just the webcam bubble - be dragged anywhere on screen, and shrinks its buttons/padding for a smaller footprint.
Alfredoalv13
pushed a commit
that referenced
this pull request
Jul 30, 2026
New _documentation/BLURBOX_INTEGRATION_TESTING.md: manual QA script covering both apps together (pause/resume during recording, both BlurBox creation paths, accept/reject/undo/export), explicitly calling out that there's no realistic automated cross-app e2e without significant new shared infra between two independently-built repos. New _documentation/BLURBOX_INTEGRATION_CONTRACT.md: VybeClip-side mirror of BlurBox's 26_VybeClip_Integration_Contract.md — file location, schema, correlation approach and its scope limitation, versioning policy. New _documentation/fixtures/redaction-events.sample.jsonl: a hand-written schema-valid fixture (both creation paths, multiple overlays, one removed with no move) for exercising the review UI without BlurBox installed. Its timestamps need shifting to align with a real test recording — the testing doc includes a regeneration one-liner. getBlurBoxEventLogPath() now honors a RECORDLY_BLURBOX_EVENT_LOG_PATH_OVERRIDE env var (same naming convention as RECORDLY_UPDATER_LOG_PATH in electron/updater.ts) so the fixture above can actually be pointed at, still only on macOS regardless of the override — matching production behavior. Two new tests cover the override on darwin and its correct no-op on other platforms. This completes the plan (BlurBox #22, VybeClip #30) — both PRs are ready for review and merge. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Test plan
npx tsc -p tsconfig.json --noEmit— cleannpx vitest --run— 819/819 passing/Applications/VybeClip.appcliclickagainst the installed app during an active screen recording:innerWidth/innerHeightmatch the display, not the old 860×160 band)barRecording/barStateViewportRecordingclasses applied, ~347×50px pill)