CaptainFact/captain-fact

Adapt background sound volume based on video volume

Ouverte

#93 ouverte le 1 avr. 2019

Voir sur GitHubGuide contributeur
 (9 commentaires) (0 réaction) (0 personne assignée) (7 forks)auto 404
bountybounty > 30€complexity > simplefrontendgood first issue

Métriques du dépôt

Stars
 (71 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

We should retrieve the volume from ReactPlayer and adapt background notifications volume accordingly. As of today the volume can be too strong or too low.

  • Store the volume from ReactPlayer in redux state
  • Retrieve the value in app/components/App/BackgroundNotifier.jsx
  • Set audiofile.volume = ___ for all notifications types (refute/confirm/neutral)

Guide contributeur

Direction de recherche
Explore comment ReactPlayer expose le volume via onProgress ou getCurrentTime ; puis stockez le volume dans Redux et utilisez le dans BackgroundNotifier pour définir audiofile.volume pour chaque type de notification.
Stack technique
reactjavascript
Domaine
frontend
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-2 jours
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
ReactReduxGit
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60