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

Disable Video button for slides not in videos #17

Open
marcerich opened this issue Oct 30, 2022 · 1 comment
Open

Disable Video button for slides not in videos #17

marcerich opened this issue Oct 30, 2022 · 1 comment
Labels
enhancement New feature or request

Comments

@marcerich
Copy link
Collaborator

We currently always display the video button if we have accompanied video content to a slide set. However, often it is the case that one adds an additional slide here and there without recording a complete new video to go along. Use-cases, e.g., think addition of quizzes, literature, home work like reading etc. in existing slide sets. In these cases the button is still shown but will restart the video from the very start, which is a severe usability problem since the users reading the slides are then lost and have to navigate back to the slide they were before pressing the video button. This navigation is possible using our internal menu if they know the title/ID of there last read, which might be tricky, and - depending on our yaml settings - the browser's back button doesn't do the trick since it might throw users to the index page. Hence, there a four general options in case of late content additions imho:

  1. Disable the button if the slide ID is missing in the *-times.json
  2. Add an additional entry in the *-times.json pointing to the slide just after the new on
  3. Add an additional entry in the *-times.json pointing to the slide just before the new on
  4. Add an additional entry in the *-times.json pointing to a new video content just showing this slide and sliced into the existing one for 10 seconds so.

Option 1. is straight forward and possibly the quick way to go. Specifically option 4 also has its benefits but needs a lot of new logic behind, e.g., updating the *-times.json from the point of entry and including the 10s offset for all slides to come and creating an artificial video using, e.g., ffmpeg (maybe with some generic audio stating the missing explanation). While 1. looks straight forward, it - as with options 2. and 3. - comes with the problem in the use-case that students work through the content first by watching the videos. Here, we might have students miss these new slides completely (bad in any case but specifically if these new slides define homework). In that use-case, I guess only option 4. does the trick kind of completely. However, that is up to decide on the severeness of the stated problem and the costs for implementing 4.

@marcerich marcerich added the enhancement New feature or request label Oct 30, 2022
@monofon
Copy link
Contributor

monofon commented Apr 13, 2023

I like option 1 the most.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants