这是indexloc提供的服务,不要输入任何密码
Skip to content

fix: handle screen share termination from browser UI #75

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Karroch-a
Copy link

Screenshot 2025-02-26 at 21 51 37
When screen sharing is stopped from the browser's native UI controls (rather than the app's control buttons), the application now properly:

  • Detects stream termination via MediaStreamTrack 'ended' event
  • Cleans up all active streams
  • Updates UI state accordingly
  • Prevents black screen artifacts

Technical changes:

  • Added MediaStreamTrack event listeners in useEffect
  • Improved stream cleanup with isStreaming check
  • Added proper TypeScript types for event handlers

When screen sharing is stopped from the browser's native UI controls
(rather than the app's control buttons), the application now properly:

- Detects stream termination via MediaStreamTrack 'ended' event
- Cleans up all active streams
- Updates UI state accordingly
- Prevents black screen artifacts

Technical changes:
- Added MediaStreamTrack event listeners in useEffect
- Improved stream cleanup with isStreaming check
- Added proper TypeScript types for event handlers
Copy link

Important

The terms of service for this installation has not been accepted. Please ask the Organization owners to visit the Gemini Code Assist Admin Console to sign it.

Copy link

google-cla bot commented Feb 26, 2025

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

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