Skip to content
This repository has been archived by the owner on May 28, 2024. It is now read-only.

Add hotkeys support for the video player #115

Closed
wants to merge 1 commit into from

Conversation

KiritoStudio
Copy link

@KiritoStudio KiritoStudio commented Mar 2, 2019

The videojs-hotkeys(https://github.com/ctd1500/videojs-hotkeys) can be used as the plugin of video.js. It works perfect. Please merge it.

@@ -150,6 +156,7 @@ var videoViewer = {
videoViewer.showPlayer();
},
showPlayer: function () {
import(/* webpackChunkName: "videojs-hotkeys" */ 'videojs-hotkeys')
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you might want to await both imports, right?

Combine them with

Promise.all([
  import(...),
  import(...),
]).then(...)

@blizzz blizzz added this to the Nextcloud 25 milestone Apr 21, 2022
@blizzz blizzz modified the milestones: Nextcloud 25, Nextcloud 26 Sep 22, 2022
@juliusknorr juliusknorr removed their request for review October 26, 2022 20:10
@blizzz blizzz modified the milestones: Nextcloud 26, Nextcloud 27 Mar 15, 2023
@blizzz
Copy link
Member

blizzz commented Mar 15, 2023

closing as files_videoplayer is not being developed further.

@blizzz blizzz closed this Mar 15, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants