casesandberg/react-color

document the onChangeComplete delay and make it configurable

Open

#516 geöffnet am 11. Juni 2018

Auf GitHub ansehen
 (7 Kommentare) (4 Reaktionen) (0 zugewiesene Personen)JavaScript (11.679 Stars) (946 Forks)batch import
enhancementhelp wanted

Beschreibung

I couldn't find any explanation in the documentation as to what onChangeComplete actually does. The code reveals that it is a hardcoded debounce with 100ms.

I suggest:

  1. Document the behavior
  2. Make the 100ms configurable

Contributor Guide

Tech Stack
reactjavascript
Domain
frontend
Issue Type
feature
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.
under 1 hour
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
stale
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
clear
Voraussetzungen
ReactJavaScriptUnderstanding of debounce
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
90
Research-Richtung
The issue points to src/components/common/ColorWrap.js line 30 where a hardcoded 100ms debounce is used. To implement, add a new prop (e.g., `onChangeCompleteDelay`) to accept a custom delay, defaulting to 100ms, and update the documentation accordingly. Check the comments for any maintainer feedback on naming or behavior.