Although this is valid Canvas operation the following results in the error "Attempted

For whoever is interested, this is a quick hotfix for the for <code class="notranslate

Made some headway on this: <a class="commit-link" data-hovercard-type="commit" dat

Transformation order about canvas2svg HOT 3 OPEN

gliffy commented on May 14, 2024 1

Transformation order

from canvas2svg.

Comments (3)

jiayihu commented on May 14, 2024 1

For whoever is interested, this is a quick hotfix for the for ctx.save() issue with Paths:

const save = Context.prototype.save;
Context.prototype.save = function () {
  /**
   * @PATCH Keep the path as `__currentElement` so that `ctx.fill()` triggering `__applyCurrentDefaultPath`
   * won't error
   */
  if (this.__currentElement.nodeName === "path") {
    !this.prevented && (this.prevented = 0);
    this.prevented += 1;
    return;
  }

  save.call(this)
};

const restore = Context.prototype.restore;
Context.prototype.restore = function () {
  /** @PATCH Check if this `ctx.restore()` was prevented */
  if (this.__currentElement.nodeName === "path" && this.prevented) {
    this.prevented -= 1;
    return;
  }

  restore.call(this)
};

from canvas2svg.

mudcube commented on May 14, 2024

Made some headway on this:
mudcube@c7df7fa

from canvas2svg.

Zamiell commented on May 14, 2024

I'm still getting the dreaded "Attempted to apply path command to node g" error, even on the latest version on master.

Apparently, the error occurs whenever you perform a ctx.save() immediately before a ctx.fill().
e.g.

const { document } = (new JSDOM('')).window;
const ctx = new C2S({
    document,
    width: 500,
    height: 500,
});
ctx.beginPath();
ctx.rect(20, 20, 150, 100);
ctx.fillStyle = "red";
ctx.save();
ctx.fill();

It will error out on the fill.
Also note that it isn't a node issue, because I get the exact same error when trying in a browser.
Does anyone have any ideas or workarounds?

from canvas2svg.

Recommend Projects