className value not escaped when set using constructor
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 42/100
- Tipo de issue
- Error
- Claridad
- Bastante claro
- Estado de actividad
- Estancado
- Stack tecnológico
- javascript
- Área
- tooling
Línea de trabajo
Reproduce the two className examples from the issue and compare constructor assignment with later value mutation. Trace the className construction and stringification paths to determine where escaping differs; done means both ways produce the expected escaped selector output, with coverage for the reported case.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Not sure if this is by design or not but I expected after #133 that class names would be escaped by default when creating a new className node.
const classNameA = parser.className({ value: 'hover:bg-blue' })
classNameA.toString()
// => '.hover:bg-blue'
const classNameB = parser.className()
classNameB.value = 'hover:bg-blue'
classNameB.toString()
// => '.hover\\:bg-blue'
Maybe @chriseppstein would know if this is intended or not? Seems weird to me to have to set the value through mutation in order to get the escaping behavior.
- Lenguaje dominante
- JavaScript
- Estrellas
- 216
- Forks
- 67
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.
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.
Más de postcss/postcss-selector-parser
-
Using @csstools/css-tokenizerAbierto
Dificultad 5/5 Más de una semana Aptitud para principiantes 25/100
-
Dificultad 3/5 1-2 días Aptitud para principiantes 74/100
-
[tentative] Add support for `.foo-*` class prefix selectorsPosiblemente ocupada @MoOx la tomó hace 60 días. Abierto
postcss/postcss-selector-parser#332 · 1 reacción · 2 asignados ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 45/100
postcss/postcss-selector-parser#309 · 1 comentario ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 45/100
postcss/postcss-selector-parser#306 · 2 comentarios ·
Todos los issues de postcss/postcss-selector-parser
Issues similares
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 92/100
QuantEcon/lecture-python-programming#642 ·
Los mantenedores suelen responder en 1 día
-
Missing repro Platform: Android
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
software-mansion/react-native-reanimated#10816 · 2 comentarios ·
Los mantenedores suelen responder en 1 día
-
[Suggestion]: Document that useFormStatus works with a preventDefault-ed onSubmit + startTransitionAbiertotype: documentation
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 2 días
-
area:docs bug triage:confirmed
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
Cotal-AI/Cotal#2875 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
keithamus/css-minify-tests#304 ·
Los mantenedores suelen responder en 1 día