videojs/video.js

Volume/Mute State Not Preserved Switching Between HTML5 and Flash Techs

Open

#2 066 ouverte le 23 avr. 2015

Voir sur GitHub
 (2 commentaires) (0 réactions) (0 assignés)JavaScript (7 502 forks)batch import
confirmedgood first issuepinnedunclaimed

Métriques du dépôt

Stars
 (39 742 stars)
Métriques de merge PR
 (Merge moyen 111j 10h) (3 PRs mergées en 30 j)

Description

I see 2 problems with volume/mute handling in videojs:

  1. Html5 and Flash tech are inconsistent handling volume and muted. Html5 maintains 2 separate variables to hold the value for volume() and muted(). Flash on the other hand sets volume = 0 when muted(true) is called. My guess is Html5 is the proper implementation desired.
  2. The state stored for volume() and muted() are delegated to the Tech. Switching between Techs causes volume and muted states to be lost between the time unloadTech and loadTech is called. A simple hack is to add a couple lines inside loadTech to save the state of muted and volume and then reapply those states inside the techReady function. Going forward, I feel the Player core should hold these states.

I'm working off a modified version of v4.12.5

Guide contributeur

Direction de recherche
Examinez la gestion de l'état du volume et de la sourdine dans les technologies HTML5 et Flash. Révisez la logique de changement de technologie dans player.js, en particulier les méthodes unloadTech et loadTech. Proposez une solution pour stocker ces états dans le cœur du Player plutôt que de les déléguer à la technologie.
Stack technique
javascript
Domaine
frontend
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Git
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60