Improve performance by running canvas.renderAll only once per frame

[thani-sh]

Hi,

I'm checking out fabricjs these days for a project we're planning to do. While experimenting with Fabric and Chrome DevTools, I came across following.

It appears that canvas.renderAll is running multiple times per browser render frame. It's triggered by the mouse event. I tried adding the following hack and the frame rate increased a bit.

let isRendering = false;
const render = canvas.renderAll.bind(canvas);

canvas.renderAll = () => {
    if (!isRendering) {
        isRendering = true;
        requestAnimationFrame(() => {
            render();
            isRendering = false;
        });
    }
};

Is it possible to improve FabricJS performance when using mouse events this way? Thanks.

[asturur]

yes we have already a ticket for this that is about implememting requestAnimationFrame

[asturur]

duplicate of #1979

[asturur]

is something i have to do anyway soon