Skip to content

Use video_player_web#170

Merged
johnpryan merged 1 commit intoflutter:masterfrom
cbenhagen:patch-1
Dec 5, 2019
Merged

Use video_player_web#170
johnpryan merged 1 commit intoflutter:masterfrom
cbenhagen:patch-1

Conversation

@cbenhagen
Copy link
Copy Markdown
Contributor

No description provided.

@cbenhagen
Copy link
Copy Markdown
Contributor Author

/cc @johnpryan

@johnpryan
Copy link
Copy Markdown
Contributor

Thanks, this looks good to me, I'll need to check the version of the Flutter SDK to use before we can put this up on https://flutter.github.io/samples/

@cbenhagen
Copy link
Copy Markdown
Contributor Author

I don't think there would be an upgrade of the Flutter SDK needed for the video_player_web plugin. But I guess I should add video_player_platform_interface to the pubspec manually or by running flutter update-packages --force-upgrade right?

@johnpryan
Copy link
Copy Markdown
Contributor

I don't think video_player_platform_interface needs to be explicitly added to the pubspec. There's just a question around which version of the Flutter SDK will work with this package.

@cbenhagen
Copy link
Copy Markdown
Contributor Author

About any recent Flutter SDK version should work with that package. It's not using anything new as far as I know. Maybe @hterkelsen can confirm?

@johnpryan johnpryan merged commit 231a19a into flutter:master Dec 5, 2019
@cbenhagen cbenhagen deleted the patch-1 branch December 5, 2019 22:26
@cbenhagen
Copy link
Copy Markdown
Contributor Author

cbenhagen commented Dec 9, 2019

@johnpryan has this been published to https://flutter.github.io/samples/#/ already?

@johnpryan
Copy link
Copy Markdown
Contributor

Yeah, although I wasn't sure if this was meant to fix the GIFs or the videos. The videos don't seem to be working.

@cbenhagen
Copy link
Copy Markdown
Contributor Author

It was supposed to fix the videos but I should have tested this locally. There is code in video_demo.dart which will not work on the web. Is it ok to fix it in this repo or do I need to fix this in https://github.com/flutter/flutter/tree/master/examples/flutter_gallery?

@johnpryan
Copy link
Copy Markdown
Contributor

I would recommend fixing it in the flutter repo, this version of the gallery is pulled from there.

@harryterkelsen
Copy link
Copy Markdown
Contributor

If this is due to videos loaded from assets, I filed an issue to support that here: flutter/flutter#46621

I will send a PR out shortly.

@cbenhagen
Copy link
Copy Markdown
Contributor Author

Thanks Harry! This is one of the issues. Others include missing connectivity plugin and usage of dart:io to check if it is running in an iOS simulator. I will search for a solution and open a PR in flutter/flutter.

@ditman
Copy link
Copy Markdown
Member

ditman commented Dec 10, 2019

I have opened a PR in flutter/flutter that should fix video playback on web. It's linked to @cbenhagen 's ticket above:

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.

4 participants