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

SPIKE/autoplay next video #54

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

SPIKE/autoplay next video #54

wants to merge 1 commit into from

Conversation

yshmarov
Copy link
Owner

Description

if opens a video -> autoplay it. when video ends, will redirect & autoplay next random video.

todo:

  • do it in a turbo frame request so that only part of page changes
  • separate template for the video to be max screen size (can not make full screen without users permission)
  • black background on the template?
  • play videos in an order, not random?

How to test

Self-check

  • self-reviewed
  • added tests
  • focused (related only to the ticket scope)

Visual Changes

Before

After

@yshmarov yshmarov self-assigned this Oct 26, 2022
@yshmarov yshmarov marked this pull request as draft October 26, 2022 17:36
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.

1 participant