fortune-sheet icon indicating copy to clipboard operation
fortune-sheet copied to clipboard

enable onOp option to capture the changes for the specific columns

Open gudipudipradeep opened this issue 11 months ago • 2 comments

Enable onOp to capture the changes of specific column instead of all columns.

async onOp(op: Op[]){ console.log(this.instruments); console.log(JSON.stringify({ req: "op", data: op })); };

enable the specific columns improve the number of iteration inside the code and improve performance. instead of hitting all columns to identify.

gudipudipradeep avatar Jul 29 '23 08:07 gudipudipradeep

I tried with extending library, but not able extend the functionality due to const. image

this.workbookRef.emitOp = this.emitOp.bind(this);

async emitOp(ctx, patches, options, undo) { console.log("++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++---------------"); };

gudipudipradeep avatar Aug 01 '23 17:08 gudipudipradeep

@shuanhukexuejia Please can you help on extending the functionality of emitOp . so that we can capture specific column or row changes.

so that number iteration will be reduced.

gudipudipradeep avatar Sep 01 '23 14:09 gudipudipradeep

this is a non-issue, and applies to a specific use case. it should be left to the user implementation to ignore an op on the rows/cols then want.

sanchit3008 avatar Mar 15 '24 18:03 sanchit3008