Invalid typing probably?
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 45/100
- Tipo de issue
- Error
- Claridad
- Bastante claro
- Estado de actividad
- Estancado
- Stack tecnológico
- javascript
- Área
- developer-experience, tooling
Línea de trabajo
Empieza comparando las definiciones de EventContext y label en types/element.d.ts y types/label.d.ts; después, inspecciona el fixture de prueba referenciado en test/fixtures/ellipse/label-dynamic.js. Comprueba si EventContext.element.label.options permite las propiedades utilizadas por los ejemplos, especialmente content; se considera terminado cuando las definiciones de tipos coinciden con el uso establecido y las comprobaciones de tipos relevantes pasan.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Hello again!
I saw lot's of usage of EventContext.element here and there in examples and tests.
e.g. [here](https://github.com/chartjs/chartjs-plugin-annotation/blob/ebd5c6f9c2f56dcf1cf213aa7af2a9e93d38059f/test/fixtures/ellipse/label-dynamic.js#L34C1-L35C1:
element.label.options.display = true;
But in typings I see that label is marked as AnnotationElement here.
Idk, did I understand it correctly, but I believe it should be CoreLabelOptions or smth similar?
I believe typing checker or eslint passes successfully because lot's of properties have the same name. But if you try to replace the code in the example above to:
element.label.options.content= "new label text";
you'll see an error since element.label.options is not type of AnnotationOptions
But maybe that's just things are mixed up in my head a bit 🙂
- Lenguaje dominante
- JavaScript
- Estrellas
- 623
- Forks
- 377
- Merge medio
- 11 h 20 min
- PR fusionados (30 d)
- 7
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Sin plantilla de pull request
- Leer la 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.
Más de chartjs/chartjs-plugin-annotation
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 48/100
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 35/100
-
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
-
TypeScript type issuesPosiblemente ocupada @joshkel la tomó hace 439 días. Abierto
Dificultad 4/5 3-5 días Aptitud para principiantes 38/100
Todos los issues de chartjs/chartjs-plugin-annotation
Issues similares
-
`yarn vitest:update` (documented) throws locally; local Cypress scripts target an unserved portAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
rescript-lang/rescript-lang.org#1415 ·
Los mantenedores suelen responder en 2 días
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
Deepak3699/Ai_Mentor#244 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
antithesishq/bombadil#361 ·
Los mantenedores suelen responder en 1 día
-
ungroomed
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
dequelabs/axe-core#5455 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 88/100
DietrichGebert/ponytail#1072 ·
Los mantenedores suelen responder en 3 días