Microsoft/TypeScript

Wrong createElementNS() type definitions

Chiusa

#24.269 aperta il 19 mag 2018

Vedi su GitHubGuida contributor
 (5 commenti) (0 reazioni) (0 assegnatari)TypeScript (13.395 fork)batch import
BugDomain: lib.d.tsHelp Wanted

Metriche repository

Star
 (108.860 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

Document.createElementNS() allows to create a few nonexistent SVG elements. In particular, this function has signatures that allow creating componentTransferFunction, textContent and textPositioning, but these correspond to SVG interfaces which do not have any real element associated.

This can be easily verified by running:

document.createElementNS('http://www.w3.org/2000/svg', 'textContent') instanceof SVGTextElement; // false
document.createElementNS('http://www.w3.org/2000/svg', 'text') instanceof SVGTextElement; // true

(Note that SVGTextElement implements SVGTextContentElement.)

Guida contributor

Direzione di ricerca
Il problema è che `Document.createElementNS()` ha overload che consentono la creazione di elementi SVG come `componentTransferFunction`, `textContent` e `textPositioning` che non esistono in SVG. La correzione comporta la rimozione di questi overload errati dalle definizioni dei tipi DOM di TypeScript. Il file rilevante è `lib.dom.d.ts` nel repository TypeScript. Cerca le firme del metodo `createElementNS` e verifica quali elementi sono validi rispetto alla specifica SVG. Controlla i commenti esistenti per eventuali contesti o suggerimenti aggiuntivi.
Tech stack
typescript
Dominio
frontend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
TypeScriptSVG DOM knowledge
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70