toSVG() ignores objects with fill type pattern with patternTransform?

[KristofferBerg]

Version

3.5.1

Test Case

http://jsfiddle.net/BrandApp_io/3zk9njxb/48/

Information about environment

Browser: Chrome Version 78.0.3904.70 (Official Build) (64-bit) - on Ubuntu 19

Steps to reproduce

I'm trying to create a "image crop functionality" (a bit like this thread #4606) in my application, and using the fill type: "pattern" on an object (rectangle is used in the JsFiddle), and then further adjust the pattern image with the offsetX and offsetY and scaling with the patternTransform parameters.

Expected Behavior

toSVG() should handle the patternTransform

Actual Behavior

toSVG() seems to ignore patternTransform?

[asturur]

I would really suggest you to use the cropX, cropY, width/height functionality if you are cropping with RECTS only.

The svg bug, if reproducible will be fixed anyway.

[asturur]

no trace of patternTransform in the pattern.toSVG code. i think we missed it completely.

[KristofferBerg]

@asturur Thanks. Yeah, but I don't want it to be limited to RECTS only :) My cropping functionality works perfectly with all kinds of fancy objects now, but it's the SVG export that suddenly was a problem here :/ (actually - I'm using the SVG to export the canvas to vector, that a PDF generator function is using. Using Chrome Puppeteer to print the SVG to PDF).

If anyone has another / safer way to export the canvas to PDF, I'm super interested in knowing how :) - in vector, that is. Not pixels.

[asturur]

probably node-canvas would be good to.
Never did it.

[Akashdg]

@asturur What is the status?, I really want svg with patternTransform in it, I am using pattern in Textbox.