Microsoft/TypeScript

Wrong createElementNS() type definitions

Open

#24.269 geöffnet am 19. Mai 2018

Auf GitHub ansehen
 (5 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)TypeScript (6.726 Forks)batch import
BugDomain: lib.d.tsHelp Wanted

Repository-Metriken

Stars
 (48.455 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 6T 17h) (9 gemergte PRs in 30 T)

Beschreibung

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.)

Contributor Guide

Research-Richtung
Das Problem ist, dass `Document.createElementNS()` Überladungen hat, die die Erstellung von SVG Elementen wie `componentTransferFunction`, `textContent` und `textPositioning` erlauben, die in SVG nicht existieren. Die Korrektur umfasst das Entfernen dieser falschen Überladungen aus den TypeScript DOM Typdefinitionen. Die relevante Datei ist `lib.dom.d.ts` im TypeScript Repository. Suchen Sie nach den Methodensignaturen von `createElementNS` und überprüfen Sie anhand der SVG Spezifikation, welche Elemente gültig sind. Überprüfen Sie vorhandene Kommentare auf zusätzlichen Kontext oder Vorschläge.
Tech Stack
typescript
Domain
frontend
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Unter 1 Stunde
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
TypeScriptSVG DOM knowledge
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70