rasterizeHTML.js icon indicating copy to clipboard operation
rasterizeHTML.js copied to clipboard

This should be bundled as ESM

Open sgarfinkel opened this issue 11 months ago • 5 comments

Currently this library is bundled as UMD. It would be very helpful if instead this used conditional exports to emit ESM and CJS for tree-shaking purposes.

sgarfinkel avatar Jan 29 '25 00:01 sgarfinkel

Hello, @cburgmer. Newcomer here exploring the various options out there, and this one seems to be the most promising. However, the code's age hurts.

Do you plan to maintain this repository? If yes, what are your priorities, if you don't mind my asking?

webJose avatar Feb 17 '25 04:02 webJose

More than happy to merge PRs :) I myself am currently not using this library, so have little need for changes.

cburgmer avatar Mar 05 '25 17:03 cburgmer

How dare you, sir, asking reasonable things!?? 😄

Got it. Thanks for the response. At least we now know a contribution could go through. 👍

webJose avatar Mar 05 '25 17:03 webJose

@cburgmer what kind of build step would you be comfortable with? Would Vite's library mode be acceptable?

bernardobelchior avatar Nov 07 '25 09:11 bernardobelchior

I haven't used Vite, and haven't kept track of the last few years of the JS ecosystem. Back then when this library was created Grunt used to be the thing. It would be good for any solution to ensure that a) the tools play nicely together while they are employed, and b) a clear path for the replacement of Grunt could be ensured (for when that time comes).

That said, I trust you to make the right call.

cburgmer avatar Nov 08 '25 17:11 cburgmer