-
-
Notifications
You must be signed in to change notification settings - Fork 5
Description
i.e. trimCanvas(canvas, {duplicate: true})
.
Given the intense demand seen by the number of upvotes on szimek/signature_pad#49 (comment) it seems like it should be included by default.
I'm also reading through my own docs for the first time in a while and if you don't know how canvas or cloning works, it may be quite unintuitive.
It's only a handful of lines of code, but IIRC the reason why I didn't include it (or removed it rather) was because it introduced a dependency on canvas
, meaning it wouldn't work in Node (requires a pretty massive native dependency). If it's hidden behind an optional flag though, this shouldn't be problematic.
This is something I've been thinking of adding for a while (in my backlog, ofc), but I also happened to see https://github.com/lukechilds/merge-images#nodejs-usage today, which is another interesting workaround for Node usage.