Skip to content

Fix Peek video player showing black screen after viewing multiple videos#10

Draft
Copilot wants to merge 2 commits intomainfrom
copilot/fix-9
Draft

Fix Peek video player showing black screen after viewing multiple videos#10
Copilot wants to merge 2 commits intomainfrom
copilot/fix-9

Conversation

Copy link

Copilot AI commented May 23, 2025

Issue

When using Peek to preview video files, after viewing approximately three videos, subsequent videos would show only a black screen instead of the expected video content. The video controls would appear but the content area would remain black.

Root Cause

The MediaPlayer object in the VideoPreview control wasn't being properly reset between video previews. After viewing multiple videos, the player would get into an inconsistent state, resulting in a black screen for subsequent videos.

Solution

The fix properly disposes of and recreates the MediaPlayer object between video previews:

  1. Added a null check for VideoPreview.MediaPlayer
  2. Updated the resource management in OnPreviewerChanging method to:
    • Pause the current player
    • Clear the current source
    • Store a reference to the old player
    • Create a new MediaPlayer instance with SetMediaPlayer
    • Dispose of the old player to release resources

This ensures each video preview starts with a fresh MediaPlayer instance, preventing the black screen issue after viewing multiple videos.

Fixes #9.


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Co-authored-by: lei9444 <39758135+lei9444@users.noreply.github.com>
Copilot AI changed the title [WIP] Peek will show a black screen on videos after viewing a few videos. Fix Peek video player showing black screen after viewing multiple videos May 23, 2025
Copilot AI requested a review from lei9444 May 23, 2025 13:12
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.

Peek will show a black screen on videos after viewing a few videos.

2 participants