normalize-color accepts functional colors with surrounding junk
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- C++
- Estrellas
- 127k
- Forks
- 25.3k
- Merge medio
- 1 d 23 h
- PR fusionados (30 d)
- 4
Descripción
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
- Open the Snack: https://snack.expo.dev/rykV9Sq-NOSJg8KWXawiP
- Look at rows for
xxrgb(1, 2, 3)yy,rgb(1, 2, 3)yy, andrgba(1,2,3,0.5)extra - Observe non-null
processColorvalues and painted swatches (bug). Spaced hex#fffcorrectly 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
Guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Línea de trabajo
Empieza con los matchers de color funcionales de @react-native/normalize-colors y reproduce los casos indicados de rgb, rgba y spaced-hex usando el Snack enlazado. Se considera terminado cuando las entradas con texto basura alrededor devuelven null, mientras que los colores funcionales válidos siguen produciendo valores no nulos; el issue también enlaza el PR de corrección #58496.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- javascript, react-native
- Área
- mobile
- Tipo de issue
- Error
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Estado de actividad
- Estancado
- Claridad
- Bien especificado
- Aptitud para principiantes
- 25/100