Mike Taghavi
Mike Taghavi
@stuartmorgan @jmagman I have recreated the changes on top of federated _video_player_.
> The web changes (a noop) are OK by me. Do mention me when the time comes to land the `web` package PR! The rest of the platform owners should...
> Don't you think it makes more sense to create a plugin that sets the audio mixing mode for the app instead of tying the functionality to the video player?...
> Yea, seems like we'll have to do a breaking change. What I suggest is removing the `setCategory` call from the `initialize` message and adding a separate call to the...
@stuartmorgan If I understand it correctly, this call then, will not result in global configuration of AVAudioSession: https://github.com/flutter/plugins/blob/main/packages/video_player/video_player/lib/video_player.dart#L28 and based on the given options, the correct platform methods could be...
@gaaclarke I think the second option you have suggested might work better in this case, to bypass the configuration of AVAudioSession and delegate the setup to another plugin only when...
@stuartmorgan Thank you for the feedback. Alright, I will do so as recommended.
@stuartmorgan I have reimplemented the changes as suggested.
@stuartmorgan Thank you for the feedback. I make the changes.
@cyanglaz Yes, I am waiting for @stuartmorgan to clarify some of my questions in order to proceed.