videojs/v10

Bug: Ejected Skin Sample Causes Fast Refresh Warning in React

Closed

#1,484 opened on Apr 28, 2026

View on GitHub
 (1 comment) (0 reactions) (1 assignee)TypeScript (67 forks)github user discovery
docsdxgood first issuepkg:reactsiteskin

Repository metrics

Stars
 (804 stars)
PR merge metrics
 (PR metrics pending)

Description

[!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 direction
Move the createPlayer function to its own separate file to avoid the React Fast Refresh warning.
Tech stack
typescriptreact
Domain
frontend
Issue type
Bug
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
1
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
Under 1 hour
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
Active
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
Clear
Prerequisites
React
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
95