feat: Add type annotations for TiledPixiTrack and utils

[etowahadams]

Towards #1172

• types for TiledPixiTrack.js
• types for utils/pixi-text-to-svg.js
• types for utils/svg-line.js

Description

What was changed in this pull request?

Why is it necessary?

Fixes #___

Checklist

• Set proper GitHub labels (e.g. v1.6+, ignore if you don't know)
• Unit tests added or updated
• Documentation added or updated
• Example(s) added or updated
• Update schema.json if there are changes to the viewconf JSON structure format
• Screenshot for visual changes (e.g. new tracks or UI changes)
• Updated CHANGELOG.md

[etowahadams]

TiledPixiTrack is 95% of the way there. There are a few gnarly areas I could use some help on though. Perhaps we can discuss on Thursday or find a time before then.

[etowahadams]

Since classes which extend this one call super with the options which later gets set to this.options, I need a template of an object which must have all of the PixiTrackOptions but can have any other additional property

[etowahadams]

One existing challenge is to get the HandleTilesetInfoFinished implementation to satisfy all three types

higlass/app/scripts/types.ts

Lines 131 to 135 in 5b32ce6

	export type HandleTilesetInfoFinished = {
	(info: null): void;
	(info: TilesetInfo, tilesetUid?: string): void;
	(error: { error: string }): void;
	};

[manzt]

One existing challenge is to get the HandleTilesetInfoFinished implementation to satisfy all three types

Agreed. It seems like that the usage of that function has been overloaded over time. Overloads are annoying to implement in TypeScript and often require lying to the typechecker. I wonder if a more flexbile definitoin would be easier to implement:

 export type HandleTilesetInfoFinished = { 
    (info: TilesetInfo | null | { error: string }, tilesetUid?: string): void; 
 }; 

[manzt]

Looking forward to reviewing this, but won't be able to until after this weekend. Let me know if you are blocked

[etowahadams]

No problem, have a good weekend!