[6.x] Asset video thumbnail generation

[JohnathonKoster]

This PR adds the ability to automatically generate thumbnails for video assets within the Control Panel's asset browser.

Docs to come!

Closes statamic/ideas#1120.

[daun]

Nice! Would it be possible to merge the hook data into the asset resource after merging the thumbnail data? That would be a simple way of allowing what PR #10594 was trying to accomplish, just via hooks. With the current order, there's no way of setting a custom thumbnail value for other asset types (e.g. PDFs) since the thumbnails method always returns ['thumbnail' => null] for non-image/video types and overwrites whatever was returned from the hook. Switching the order and merging the hook data after the thumbnail data would make that possible.

cms/src/Http/Resources/CP/Assets/FolderAsset.php

Lines 89 to 90 in a90c7d2

	$this->mergeWhen(! empty($hookData), $hookData),
	$this->merge($this->thumbnails()),

Or maybe add another explicit thumbnail hook, if that makes more sense?

[JohnathonKoster]

Nice! Would it be possible to merge the hook data into the asset resource after merging the thumbnail data? That would be a simple way of allowing what PR #10594 was trying to accomplish, just via hooks. With the current order, there's no way of setting a custom thumbnail value for other asset types (e.g. PDFs) since the thumbnails method always returns ['thumbnail' => null] for non-image/video types and overwrites whatever was returned from the hook. Switching the order and merging the hook data after the thumbnail data would make that possible.

cms/src/Http/Resources/CP/Assets/FolderAsset.php

Lines 89 to 90 in a90c7d2

	$this->mergeWhen(! empty($hookData), $hookData),
	$this->merge($this->thumbnails()),

Or maybe add another explicit thumbnail hook, if that makes more sense?

Love this! I've made an adjustment to how things are merged that will make it possible to override any thumbnail 🤘

[daun]

Amazing! Can't wait for this to land 🎊