Type error setting pointStype to false
- Langage dominant
- JavaScript
- Étoiles
- 67.7k
- Forks
- 11.9k
- Merge moyen
- 7 h 39 min
- PR mergées (30 j)
- 5
Description
### Expected behavior
I hope I can define a dataset with a boolean pointStyle without getting a typing error
### Current behavior
When i add the following config:
```typescript
new Chart(canvas, {
type: 'line',
data: {
labels: generateLabels(),
datasets: [
{
label: 'Dataset',
data: generateData(),
pointStyle: false, <--
},
],
}
```
I get the following typing error
_Type '{ pointStyle: boolean; }' is not assignable to type 'ChartDatasetCustomTypesPerDataset<"line", number[]>'._
### Reproducible sample
https://codesandbox.io/s/ng2charts-chart-js-issue-template-forked-r5kcm4
Updated example: https://codesandbox.io/s/ng2charts-chart-js-issue-template-forked-6fd62d?file=/src/app/app.component.ts
(does not complain about the boolean type, but neither does it complain about an invalid string type)
### Optional extra steps/info to reproduce
_No response_
### Possible solution
It is currently "fixed" by forcing a type as shown here
`pointStyle: false as unknown as string,`
but I wouldn't want to have to do this.
### Context
I am working on Angular 16
### chart.js version
4.3.0
### Browser name and version
Firefox Developer Edition 115.0b9 (64-bit) on Ubuntu
### Link to your project
_No response_
Guide de contribution
Aucun guide de contribution indexé pour ce dépôt
Piste de recherche
Start with the Chart.js TypeScript dataset configuration for a line chart and trace the declared type of the pointStyle property. Use the updated CodeSandbox to verify that false is accepted while an invalid string remains rejected; done means the reported typing error is resolved without the workaround.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- javascript, typescript
- Domaine
- frontend
- Type d'issue
- Bug
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Activité
- À l'abandon
- Clarté
- Plutôt claire
- Accessibilité débutants
- 42/100