lbryio/lbry-desktop

The mute icon sometimes does not reflect actual state.

Open

#4.358 geöffnet am 9. Juni 2020

Auf GitHub ansehen
 (0 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (412 Forks)batch import
area: viewerhelp wantedtype: bug

Repository-Metriken

Stars
 (3.596 Stars)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

Bug The mute icon in the player does not reflect the actual state for this scenario. It's very minor, but would probably impact the "tap to unmute" feature request.

To Reproduce

  1. Open a video and mute it.
  2. Click Back and close the floating player.
  3. Open another video, and click to play (I had "autoplay" set to off). It should then start as muted, and if not, repeat 1 and 2 again.
  4. Press 'm' to unmute. You'll hear the sound, but icon stays as "muted". You might need to wiggle the mouse to keep the controls visible.

If you enter into a video claim with audio played directly, then the 'm' button is in sync with the icon when pressed.

Expected behavior Icons need to reflect actual state.

System Configuration 0.46.1 yarn build, but also seen in lbry.tv today.

Contributor Guide

Research-Richtung
Verfolgen Sie die Stummschaltungszustandsverwaltung in der Player Komponente. Suchen Sie nach der Stelle, an der das Stummschaltungssymbol aktualisiert wird und wo die Audio Stummschaltung ausgelöst wird. Untersuchen Sie, ob es eine Race Condition oder eine fehlende Zustandssynchronisation beim Navigieren zwischen Videos gibt.
Tech Stack
javascript
Domain
desktop
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Frisch
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
GitJavaScript
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
80