ElemeFE / ElemeFE/element-react

Issues about TypeScript definition

Aperta
#697 1 commento 1 reazione 0 assegnatari Vedi su GitHub
Lingua principale
JavaScript
Stelle
2.8k
Fork
435
Metriche di merge delle PR
Nessuna PR unita negli ultimi 30g

Descrizione

### Description

TypeScript definitions of many components are wrong.

### Reproduce Steps

For example, in [definition of `Input` component](https://github.com/eleme/element-react/blob/1af5231247b5b1b097a5135cf7d0c3e45b50c629/typings/index.d.ts#L437), you write
```
onChange?(e?: React.SyntheticEvent): void
```

in the real world, the component pass the string (value of input) to the `onChange` method, which should be
```
onChange?(value: string): void
```

Another case, `emptyText` is not present in attributes of `Table` components, which is present in your docs.

### Solution

Hopes there will be **a full check of all definition** and fix in TS .d.ts file. Or I may bring a pull request to fix the issues I ran into (which might be not complete)

Guida per i contributori

Apri la guida per i contributori

Direzione di ricerca

Inizia in typings/index.d.ts, vicino alla definizione di Input, e confronta la sua firma onChange con il comportamento del componente descritto nell’issue. Verifica i typings di Table rispetto alla documentazione di emptyText, quindi esamina le altre definizioni per individuare discrepanze simili. Il lavoro è terminato quando le dichiarazioni .d.ts interessate corrispondono alle API documentate ed effettive dei componenti.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
react, typescript
Ambito
frontend
Tipo di issue
Bug
Difficoltà
4/5
Tempo stimato
3-5 giorni
Stato di attività
Ferma
Chiarezza
Da chiarire
Idoneità per principianti
25/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.