The player's sound control and the stored "narrate by default" are one setting

Aperta
  • Qualcosa non va?
#20 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
3/5
Tempo stimato
Mezza giornata
Idoneità per principianti
66/100
Tipo di issue
Bug
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
typescript
Ambito
frontend

Direzione di ricerca

Start in DeckPlayer.tsx, where the speaker and the sheet's Narration switch currently call narration.toggle, and in src/voicePreference.tsx, which holds connected-deck:narrate-by-default. Add the optional onToggleNarration prop and have PresentationDeck pass the stored setter, so all three surfaces read and write one value. Done when a deck opened after muting starts silent, Settings shows Silent, and the engine's tests still pass.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

enhancement

Why

The viewer bar's speaker and the sheet's Narration switch change only the deck being watched (DeckPlayer.tsx passes narration.toggle). The launch page's Settings keeps a separate stored preference, connected-deck:narrate-by-default (src/voicePreference.tsx). Turn narration on inside a deck, and the next deck opens silent. Turn it off in Settings, and the deck you have open doesn't change. One question gets answered in two places that disagree.

What changes

  1. DeckPlayer takes an optional onToggleNarration. When a host passes it, narrateByDefault is the live setting, and the bar's speaker and the sheet's switch call it instead of toggling the session.
  2. PresentationDeck passes the stored preference's setter, so the speaker, the sheet and the launch page's Settings all read and write one value.
  3. Turning narration off still clears Audiobook mode.
  4. A host that passes nothing keeps today's session-only toggle.

Done when

  • Muting inside a deck and then opening another deck opens it muted, and the launch page's Settings shows Silent.
  • Turning narration on in Settings takes effect in an open deck's sheet.
  • The engine's tests pass.

Out of scope

  • The notes window's own narration toggle.
  • The sheet's layout.
Lingua principale
TypeScript
Stelle
1
Fork
0
Merge medio
30m
PR unite (30g)
9

Preparare l'ambiente

Questo progetto non fornisce container di sviluppo, Dockerfile né guida per i contributori, quindi l'ambiente è a tuo carico: parti dal suo README e consulta la nostra guida al primo contributo per i passaggi generali.

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di johnfrog76/connected-deck

Tutte le issue di johnfrog76/connected-deck

Issue simili

Altre issue su TypeScript

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.