videojs/v10

Bug: Ejected Skin Sample Causes Fast Refresh Warning in React

Geschlossen

#1.484 geöffnet am 28.04.2026

Auf GitHub ansehenContributor Guide
 (1 Kommentar) (0 Reaktionen) (1 zugewiesene Person)TypeScript (67 Forks)github user discovery
docsdxgood first issuepkg:reactsiteskin

Repository-Metriken

Stars
 (804 Sterne)
PR-Merge-Metriken
 (Durchschn. Merge 14T 19h) (55 gemergte PRs in 30 T)

Beschreibung

[!NOTE] Triage estimate — cycle priority and story points were assigned during an AI-assisted planning pass and are rough first guesses. Confirm or adjust when you pick this up.

The ejected skin sample co-locates createPlayer and the component in the same file. This causes a React Fast Refresh warning. The fix is to move createPlayer to its own file.

Fix needed in: Ejected skin sample file (around line 260 area)

Contributor Guide

Research-Richtung
Verschieben Sie die createPlayer Funktion in eine eigene separate Datei, um die React Fast Refresh Warnung zu vermeiden.
Tech Stack
typescriptreact
Domain
frontend
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
1
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Unter 1 Stunde
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
React
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
95