normalize-color accepts functional colors with surrounding junk

Aberta
#58,495 1 comentário 0 reações 0 responsáveis Ver no GitHub

Ninguém assumiu esta issue ainda.

Avaliação

Dificuldade
2/5
Tempo estimado
1-3 horas
Facilidade para iniciantes
25/100
Tipo de issue
Bug
Clareza
Claramente especificada
Status de atividade
Estagnada
Stack de tecnologia
javascript, react-native
Domínio
mobile

Direção de pesquisa

Comece com os matchers de cores funcionais em @react-native/normalize-colors e reproduza os casos listados de rgb, rgba e spaced-hex usando o Snack vinculado. Está concluído quando entradas com lixo ao redor retornarem null, enquanto cores funcionais válidas continuarem produzindo valores não nulos; a issue também contém um link para o PR de correção #58496.

Escrita pelo modelo de indexação a partir do texto da issue.

Descrição

Needs: Attention Needs: Repro
Description

@react-native/normalize-colors fully anchors hex matchers (^…$) but not rgb/rgba/hsl/hsla/hwb. Substring matches are therefore accepted:

normalizeColor('xxrgb(1, 2, 3)yy') // => 0x010203ff (unexpected)
normalizeColor('rgba(1,2,3,0.5)extra') // => non-null (unexpected)
normalizeColor(' #fff ') // => null (hex correctly rejects)
Steps to reproduce
  1. Open the Snack: https://snack.expo.dev/rykV9Sq-NOSJg8KWXawiP
  2. Look at rows for xxrgb(1, 2, 3)yy, rgb(1, 2, 3)yy, and rgba(1,2,3,0.5)extra
  3. Observe non-null processColor values and painted swatches (bug). Spaced hex #fff correctly stays null.
React Native Version

main / latest (package @react-native/normalize-colors, used by processColor)

Affected Platforms
  • Runtime - Android
  • Runtime - iOS
  • Runtime - Web
  • Other (please specify): pure JS @react-native/normalize-colors
Output of npx @react-native-community/cli info

N/A — reproducible via Snack / unit package without a local app install.

Stacktrace or Logs

N/A (incorrect non-null return value, no crash)

MANDATORY Reproducer

https://snack.expo.dev/rykV9Sq-NOSJg8KWXawiP

Screenshots and Videos

N/A

Fix PR: https://github.com/facebook/react-native/pull/58496

Linguagem predominante
C++
Estrelas
127k
Forks
25.3k
Nenhum PR com merge em 30d

Guia de contribuição

Abrir o guia de contribuição

Primeiros passos

  1. Leia a issue inteira e depois o guia de contribuição do projeto.
  2. Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
  3. Faça um fork do repositório e trabalhe em uma branch.
  4. Abra um pull request que referencie o número da issue.

Mais de react/react-native

Todas as issues de react/react-native

Issues semelhantes

Mais issues de C++

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.