freefeed-react-client icon indicating copy to clipboard operation
freefeed-react-client copied to clipboard

Play video attachments using inline video tag

Open n1313 opened this issue 4 years ago • 9 comments

This PR makes video attachments (files that end in .mov or .mp4) play using inline <video> tag. The videos do not autoplay and are muted by default. There is no lightbox support for now, and there are no useful thumbnails (this requires backend support).

Screenshot 2021-07-06 at 12 56 05 Screenshot 2021-07-06 at 12 56 31

n1313 avatar Jul 06 '21 19:07 n1313

Это пока для беты. Поставь пожалуйста DRAFT чтобы не смерджили случайно

indeyets avatar Jul 06 '21 20:07 indeyets

Поскольку превью всё равно нет, а плеер даже соотношение сторон правильное не подхватывает, может быть не выводить сразу тег video, а показывать его после клика по кнопке/ссылке/иконке “Play video”?

davidmz avatar Jul 06 '21 20:07 davidmz

@davidmz и включить автоплей?

n1313 avatar Jul 06 '21 20:07 n1313

Ну, там уже и автоплей можно. А то так выглядит немного бажно (с точки зрения простого юзера)

davidmz avatar Jul 06 '21 20:07 davidmz

Есть вариант сменить preload на metadata, и проблема целиком снимется (будет появляться тамбнейл правильного размера), но тогда от каждого видео браузер будет автоматически загружать первые несколько секунд. У меня почему-то фаерфокс не всегда показывает в сетевых запросах, что происходит, но от Диминого 14мб видео он автоматом загрузил первые 2.5мб. Разные браузеры (особенно на мобильниках) будут разное количество данных скачивать, сложно будет контролировать.

n1313 avatar Jul 07 '21 04:07 n1313

Могу предложить такой вариант: Screenshot 2021-07-06 at 21 25 55

n1313 avatar Jul 07 '21 04:07 n1313

@n1313 я сделал rebase на релизную ветку чтобы в бету было корректно забрать

indeyets avatar Jul 07 '21 07:07 indeyets

Да, как-то так. metadata не стоит использовать, мобильники его игнорируют (и правильно делают).

davidmz avatar Jul 07 '21 08:07 davidmz

:recycle: PR Preview 2eb2e24295b9d1c092aca182e5b587c8734e019a has been successfully destroyed since this PR has been closed.

🤖 By surge-preview

github-actions[bot] avatar Jan 07 '22 04:01 github-actions[bot]