search

jerch / xterm-addon-image

Image addon for xterm.js
MIT License
51 stars 6 forks source link

make worker path optional #7

Closed jerch closed 1 year ago

jerch commented 2 years ago

Postponed, as it is quite involved to get done right (might need some sort of preprocessor logic, which typescript lacks). Also see different build targets in node-sixel.

jerch commented 2 years ago

Also check for obvious perf smells in https://github.com/jerch/xterm-addon-image/issues/1#issuecomment-1030847030.

jerch commented 1 year ago

The current worker approach with a distributed ready-to-go worker file furthermore creates some frictions in projects, how to add the worker asset conveniently. Needs some investigations about proper worker file bundling, maybe we can provide an easier to digest asset, which does not collide with package versioning/bundling setups.