Wrong source and sourceIndex for Sass interpolated selectors
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 48/100
- Tipo de issue
- Error
- Claridad
- Bastante claro
- Estado de actividad
- Estancado
- Stack tecnológico
- javascript, sass
- Área
- tooling
Línea de trabajo
Start with the inline Sass selector reproduction and the parser code that assigns source positions and sourceIndex to interpolated class nodes. Update the behavior so the interpolated selector reports the expected columns and index, then verify both the Sass and normal selector examples pass.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Repro:
test('Sass selector', '.#{$classname}', (t, tree) => {
const node = tree.nodes[0].nodes[0];
t.deepEqual(node.type, "class"); // Ok
t.deepEqual(node.value, "#{$classname}"); // Ok
t.deepEqual(node.source.start.column, 1); // Fail. Received: 5
t.deepEqual(node.source.end.column, 14); // Fail. Received: 18
t.deepEqual(node.sourceIndex, 0); // Fail. Received: 4
});
Compare with this selector that has the same length:
test('Normal selector', '.XXXclassnameX', (t, tree) => {
const node = tree.nodes[0].nodes[0];
t.deepEqual(node.type, "class"); // Ok
t.deepEqual(node.value, "XXXclassnameX"); // Ok
t.deepEqual(node.source.start.column, 1); // Ok
t.deepEqual(node.source.end.column, 14); // Ok
t.deepEqual(node.sourceIndex, 0); // Ok
});
I tried to look at the parser code to figure out what's wrong but I didn't manage to figure this one out.
- Lenguaje dominante
- JavaScript
- Estrellas
- 214
- Forks
- 63
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Guía de contribución
No hay ninguna guía de contribución indexada para este repositorio
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-tokenizer Abierto
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
-
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 2/5 1-3 horas Aptitud para principiantes 65/100
-
Theme loads third-party resources on every page (jsdelivr web font, cdnjs Font Awesome) – GDPR Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
docToolchain/docToolchain#1705 ·
-
Timezone select lists one option per character; duplicate "Failed" reason; shared tracker popover id Abiertobug
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
nightscout/nocturne#1414 ·
-
bug v2
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
modelcontextprotocol/inspector#2458 · 1 comentario ·
-
Mend: dependency security vulnerability status: needs triage 🕵️♀️
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
carbon-design-system/ibm-products#9907 ·