casesandberg/react-color

Propogate view from toggle button on ChromePicker

Open

#584 aberto em 5 de fev. de 2019

Ver no GitHub
 (2 comments) (0 reactions) (0 assignees)JavaScript (11.679 stars) (946 forks)batch import
enhancementhelp wanted

Description

The current view state of the ChromeFields input in ChromePicker is not externally accessible and makes it difficult to tie in with other components.

I suggest adding an onToggleView callback to pass the new "view" state as a parameter back up to its container. e.g: if(this.props.onToggleView){ this.props.onToggleView('hex'); }

Guia do colaborador

Pilha de tecnologia
react
Domain
frontend
Tipo Issue
feature
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 hours
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
stale
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
clear
Prerequisites
React basicsreact color API
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
75
Direção de pesquisa
The issue requests adding an onToggleView callback to ChromePicker to pass the view state (e.g., 'hex') to the container. The change is localized to the ChromePicker component. Look at the existing props and how other callbacks like onChange are implemented. Examine the ChromeFields input to determine where the view state is toggled. Add a prop and invoke it with the new view value when the toggle button is clicked. No linked PRs or maintainer comments addressing this request.