Skip to content
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

New feature video cover selection #29

Merged
merged 25 commits into from
Aug 3, 2021
Merged

New feature video cover selection #29

merged 25 commits into from
Aug 3, 2021

Conversation

LeGoffMael
Copy link
Owner

This pull_request includes new widgets allowing the user to select his favorite thumbnail between an automatic selection.

I think this could be a good feature to include in this package because I haven't found any other library doing it. But it's a feature that you can find on Youtube or Instagram for example. All transformation data is also shared with these new widgets.

New feature

  • generate x amount of thumbnails from the trimmed period using video_thumbnail library.
  • synch transformation data with video and coverViewer (even with tabs)
  • be able to export the cover selected by using ffmpeg

General changes

  • new widgets CoverSelection and CoverViewer
cover_feature_example.mp4

Solution explanation

Originally, I had planned to select the cover like in the Instagram interface while using the video player, but it looks like Flutter video_player's seekTo() function is not instantaneous and does not work properly when the video is paused (#61004,#80054).

So I preferred to generate a fixed number of thumbnails each time the video is trimmed, so that the user can select his favorite thumbnail.

@seel-channel
Copy link
Collaborator

Bro, I've just merged the pull request #28 and it generated a conflict

@LeGoffMael
Copy link
Owner Author

Hey ! I merged the new master into my branch they should not be any conflict anymore

@seel-channel seel-channel merged commit 72fa07f into LeGoffMael:master Aug 3, 2021
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.

None yet

2 participants