casesandberg/react-color

Cant type in 4 or 8 letter color names in editable input anymore

Open

#552 aberto em 19 de out. de 2018

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

Description

On the react color demo page I am able to type in "blue" into any of the editable inputs and that is recognized as valid. Since this fix - https://github.com/casesandberg/react-color/commit/669bf040955d2b8ff82b491e1044d28fe80d06b5#diff-012ff5a8f8316a2507102ec918eaf81b I am not able to type in 4 or 8 letter color names. They return as invalid because of the length check. So while "red" or "yellow" works fine, "blue" or "gray" doesnt.

Is there a work around for this?

Guia do colaborador

Pilha de tecnologia
javascriptreact
Domain
frontend
Tipo Issue
bug
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.
under 1 hour
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
familiarity with Reactbasic string validation logic
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 is caused by a commit that added a length check for color names. Examine the commit at https://github.com/casesandberg/react color/commit/669bf040955d2b8ff82b491e1044d28fe80d06b5 and locate the helper function that validates color names (likely in src/helpers/). The current check rejects names with lengths other than 3, 5, 6, or 7, but should allow any length (e.g., 4 for 'blue', 8 for 'gray'). To fix, remove or modify the length validation to accept all valid CSS color names. No existing PR or assignee, so it's open for contribution.