Skip to content

fix(desktop): stop calling a cancelled share a Screen Recording problem - #193

Merged
TheCryptoDonkey merged 1 commit into
mainfrom
fix/share-cancel-not-permission
Sep 27, 2026
Merged

TheCryptoDonkey merged 1 commit into
mainfrom
fix/share-cancel-not-permission

Conversation

@TheCryptoDonkey

Copy link
Copy Markdown
Member

The owner saw "Your computer did not allow screen sharing…" on the Mac desktop app while sharing worked.

Electron 44's DisplayMediaDeviceChosen answers callback() with nothing as INVALID_DISPLAY_CAPTURE_CONSTRAINTS, the same error a missing Screen Recording permission produces. The app refuses that way whenever the person cancels or dismisses the "Choose what to share" menu, so every cancel showed the permission advice.

  • desktop:screen-access IPC (trusted sender only) returns systemPreferences.getMediaAccessStatus('screen') on macOS, granted elsewhere.
  • showShareError asks it before showing the advice; with access granted a refusal was the person and nothing is shown.

Test: portrait-share.spec gains a case with a stand-in desktop bridge (granted: no advice; denied: advice). Not yet tried in the packaged Mac app.

🤖 Generated with Claude Code

https://claude.ai/code/session_01CG4pPCsd8pdySNvBpt8fTk

Electron answers a refused display request with "Invalid capture
constraints" whatever the reason, so closing the "Choose what to share"
menu showed the Mac permission steps to somebody who could share fine.
The page now asks the desktop app for macOS's Screen Recording status
and only gives that advice when it is not granted.

Claude-Session: https://claude.ai/code/session_01CG4pPCsd8pdySNvBpt8fTk
@TheCryptoDonkey
TheCryptoDonkey merged commit 25272f1 into main Sep 27, 2026
5 checks passed
@TheCryptoDonkey
TheCryptoDonkey deleted the fix/share-cancel-not-permission branch September 28, 2026 02:10
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