Full screen events

[ne0rrmatrix]

• Feature/Proposal

Add Support for Full Screen Events for Windows, Android, iOS and Mac Catalyst.

Description of Change

Add event handlers to notify developer when full screen status changes.

API:

/// <summary>
/// Backing store for the <see cref="FullScreenState"/> property.
/// </summary>
public static readonly BindableProperty FullScreenProperty =
	BindableProperty.Create(nameof(FullScreenState), typeof(MediaElementScreenState), typeof(MediaElement), 
		MediaElementScreenState.Default, propertyChanged: OnFullScreenPropertyChanged);
/// <summary>
/// Gets the full screen state of the media element.
/// </summary>
public MediaElementScreenState FullScreenState
{
	get => (MediaElementScreenState)GetValue(FullScreenProperty);
	private set => SetValue(FullScreenProperty, value);
}

XAML:

<toolkit:MediaElement
    x:Name="MediaElement"
    ShouldAutoPlay="True"
    Source="https://commondatastorage.googleapis.com/gtv-videos-bucket/sample/BigBuckBunny.mp4"
    MetadataArtworkUrl="https://lh3.googleusercontent.com/pw/AP1GczNRrebWCJvfdIau1EbsyyYiwAfwHS0JXjbioXvHqEwYIIdCzuLodQCZmA57GADIo5iB3yMMx3t_vsefbfoHwSg0jfUjIXaI83xpiih6d-oT7qD_slR0VgNtfAwJhDBU09kS5V2T5ZML-WWZn8IrjD4J-g=w1792-h1024-s-no-gm"
    MetadataTitle="Big Buck Bunny"
    MetadataArtist="Blender Foundation"
    MediaEnded="OnMediaEnded"
    MediaFailed="OnMediaFailed"
    MediaOpened="OnMediaOpened"
    PositionChanged="OnPositionChanged"
    StateChanged="OnStateChanged"
    SeekCompleted="OnSeekCompleted"
    FullScreenStateChanged="MediaElement_FullScreenStateChanged"/>

Code Behind:

void MediaElement_FullScreenStateChanged(object? sender, FullScreenStateChangedEventArgs e) =>
	logger.LogInformation("FullScreen State Changed. Old State: {PreviousState}, New State: {NewState}", e.PreviousState, e.NewState);

Linked Issues

• Fixes [Proposal] API that allows developers to hook a media element event and know the full screen status. #1927

PR Checklist

• Has a linked Issue, and the Issue has been approved(bug) or Championed (feature/proposal)
• Has tests (if omitted, state reason in description)
• Has samples (if omitted, state reason in description)
• Rebased on top of main at time of PR
• Changes adhere to coding standard
• Documentation created or updated: https://github.com/MicrosoftDocs/CommunityToolkit/pulls/446

Additional information

This is a PR to implement a solution for adding support for developers to have an the choice to see when user goes into full full screen and back again.

Note for Maintainers: If you see a need for a name change please go ahead and edit. Names are important. I have no opinion on naming of classes/functions/variables. If you see a need for them to change feel free to make those edits.

[cvazquez08]

+1 to merge this in 😄

@ne0rrmatrix - Any plans to merge this in?

[ne0rrmatrix]

+1 to merge this in 😄

@ne0rrmatrix - Any plans to merge this in?

Yes it will be merged some time in the future. I am working on porting Media 3 and after it is done I will be getting back to this PR and a few others and making sure they are GTG and waiting on review after that. All my PR's are essentially on hold waiting for media3. Media 3 is the new home of Exoplayer and will allow us to not be on a deprecated library anymore. It is near to being ready and all the bugs have been fixed.

ATM the nugets are not ready for us. I have One PR waiting for approval. That will fix at least one show stopper bug in media 3 bindings. The current bindings also have a show stopper bug that was fixed by me in another PR that has been merged.

It will be another few weeks to maybe a month before it will be ready for review. Fingers crossed it may be merge in October if we are super lucky. Most likely November. Following that I will be looking back at this PR and others to make sure they are fine and then I will bug the team to review them.

[ne0rrmatrix]

+1 to merge this in 😄

@ne0rrmatrix - Any plans to merge this in?

This will be looked at after #2076 has merged

[cvazquez08]

+1 to merge this in 😄

@ne0rrmatrix - Any plans to merge this in?

This will be looked at after #2076 has merged

Thank you!

Copilot reviewed 5 out of 10 changed files in this pull request and generated no suggestions.

Files not reviewed (5)

• samples/CommunityToolkit.Maui.Sample/Pages/Views/MediaElement/MediaElementPage.xaml: Language not supported
• samples/CommunityToolkit.Maui.Sample/Pages/Views/MediaElement/MediaElementPage.xaml.cs: Evaluated as low risk
• src/CommunityToolkit.Maui.MediaElement/Interfaces/IMediaElement.cs: Evaluated as low risk
• src/CommunityToolkit.Maui.MediaElement/MediaElement.shared.cs: Evaluated as low risk
• src/CommunityToolkit.Maui.MediaElement/Primitives/FullScreenStateChangedEventArgs.cs: Evaluated as low risk

Comments skipped due to low confidence (3)

src/CommunityToolkit.Maui.MediaElement/Views/MediaManager.shared.cs:57

• [nitpick] The method name 'OnWindowsChanged' is misleading as it suggests it is specific to Windows. Consider renaming it to 'OnFullScreenStateChanged' to clearly indicate its purpose.

public static void OnWindowsChanged(FullScreenStateChangedEventArgs e) => WindowsChanged?.Invoke(null, e);

src/CommunityToolkit.Maui.MediaElement/Views/MauiMediaElement.android.cs:195

• [nitpick] Consider marking the CurrentPlatformContext struct as sealed instead of readonly to prevent inheritance and ensure immutability.

readonly record struct CurrentPlatformContext

src/CommunityToolkit.Maui.MediaElement/Primitives/MediaElementFullScreenState.cs:4

• The summary comments for the enum values should be more descriptive to provide better context for developers.

///