WebReflection / WebReflection/linkedom
Typescript type errors
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- HTML
- Sterne
- 2.1k
- Forks
- 104
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Beschreibung
Hi there,
I'm running into loads of type errors from linkedom when using the library in a typescript project.
These type errors appear to all be either missing properties or incorrect implementations of base class methods (due to parameter types or return types not matching).
The errors are too long to send but I will send a few:
node_modules/linkedom/types/dom/parser.d.ts:12:5 - error TS2416: Property 'parseFromString' in type 'DOMParser' is not assignable to the same property in base type 'DOMParser'.
Type '<MIME extends "image/svg+xml" | "text/html" | "text/xml">(markupLanguage: string, mimeType: MIME) => { "text/html": HTMLDocument; "image/svg+xml": SVGDocument; "text/xml": XMLDocument; }[MIME]' is not assignable to type '(string: string, type: DOMParserSupportedType) => Document'.
Type 'HTMLDocument | SVGDocument | XMLDocument' is not assignable to type 'Document'.
Type 'HTMLDocument' is missing the following properties from type 'Document': URL, alinkColor, anchors, applets, and 181 more.
12 parseFromString<MIME extends keyof {
~~~~~~~~~~~~~~~
node_modules/linkedom/types/html/anchor-element.d.ts:4:14 - error TS2420: Class 'import("C:/Users/felis/home/projects/test-linkedom/node_modules/linkedom/types/html/anchor-element").HTMLAnchorElement' incorrectly implements interface 'HTMLAnchorElement'.
Type 'HTMLAnchorElement' is missing the following properties from type 'HTMLAnchorElement': charset, coords, hreflang, name, and 132 more.
4 export class HTMLAnchorElement extends HTMLElement implements globalThis.HTMLAnchorElement {
~~~~~~~~~~~~~~~~~
...
node_modules/linkedom/types/html/document.d.ts:6:9 - error TS2416: Property 'all' in type 'HTMLDocument' is not assignable to the same property in base type 'HTMLDocument'.
Property 'namedItem' is missing in type 'NodeList' but required in type 'HTMLAllCollection'.
6 get all(): NodeList;
~~~
../../../AppData/Roaming/npm/node_modules/typescript/lib/lib.dom.d.ts:5953:5
5953 namedItem(name: string): HTMLCollection | Element | null;
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
'namedItem' is declared here.
node_modules/linkedom/types/html/document.d.ts:15:9 - error TS2416: Property 'title' in type 'HTMLDocument' is not assignable to the same property in base type 'HTMLDocument'.
Type 'HTMLTitleElement' is not assignable to type 'string'.
15 set title(arg: HTMLTitleElement);
~~~~~
...
Workaround: tsc --skipLibCheck
It'd be great to get the types correct here so --skipLibCheck isn't needed. I would make a PR for this however it seems design decisions would need to be made to fix the types. I'm not sure if the types can be reconciled while still extending / implementing globalThis.DOMParser, globalThis.HTMLElement, /globalThis\.HTML.*Element/, etc.
Beitragsleitfaden
Für dieses Repository ist kein Beitragsleitfaden indexiert
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Reproduziere die Fehler mit TypeScript und untersuche die Deklarationen unter types/dom/parser.d.ts, types/html/anchor-element.d.ts und types/html/document.d.ts zusammen mit TypeScripts lib.dom.d.ts. Vergleiche die gemeldeten geerbten und implementierten Signaturen, um festzustellen, ob die Typen mit den globalen DOM-Schnittstellen in Einklang gebracht werden können. Als erledigt gilt die Aufgabe, wenn die gemeldeten Typfehler tsc --skipLibCheck nicht mehr erfordern.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- typescript
- Bereich
- web-dev
- Issue-Typ
- Bug
- Schwierigkeit
- 5/5
- Geschätzter Aufwand
- Über eine Woche
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 25/100