You can test it by adding https://github.com/karlvr/exceljs#integration to your package.json.

I just added it to my package.json and works like a charm. Is ext compulsory?
Edit: It would be awesome to have this fix merged ASAP. I'm stuck with this problem for quite a while now.

I think you need to provide either ext or use the existing method. I guess we could allow ext to be omitted if we just used the image width and height?

@karlvr definitely. If ext is not specified, we should use the original image's dimensions. One might get the image programmatically somewhere without knowing the exact width and height. Obviously there are libraries out there that help with finding images' sizes, but still would be nice if your PR does it out of the box

Looking into this now. Looks good, my only comment so far would be to offset tl by -1, -1 like the two-cell anchor. I'll look into this

Hi @karlvr , I have an issue using this functionality - exporting images looks great but they aren't getting deleted when I deleted a row from the Excel file. When I use use the default (tl: br:) the attribute editAs: 'oneCell' resolved this issue, but it doesn't seem to work when I use tl: ext: Did anyone else came across this problem?

@karlvr I have issues when using ext for image size.

The excel says that "We found unreadable content in 'Export.xlsx'. Do you want to recovery the contents of this workbook?" After repairing the file using excel, its opening fine.

The error log says "Repaired Records: Drawing from /xl/drawings/drawing1.xml part (Drawing shape)". So i compared the drawing1.xml from both repaired and unrepaired workbooks. And found that if i remove editAs="oneCell" attribute from all xdr:oneCellAnchor tags, the error is gone.

If you set ext with height and width it works but you get the MS error and repair. I did a diff on the drawings xml of the original and repaired. No changes what-so-ever were made other than on the root xdr:oneCellAnchor where excelJS sets an attribute 'editAs="absolute"' or 'editAs="oneCell"', Microsoft wipes that out (no attribute at all). That is the only change. So very confused on what the heck we need to do to make this work. Should you not be setting this attribute? Is it defined elsewhere where MS is barfing?

If you set ext with height and width it works but you get the MS error and repair. I did a diff on the drawings xml of the original and repaired. No changes what-so-ever were made other than on the root xdr:oneCellAnchor where excelJS sets an attribute 'editAs="absolute"' or 'editAs="oneCell"', Microsoft wipes that out (no attribute at all). That is the only change. So very confused on what the heck we need to do to make this work. Should you not be setting this attribute? Is it defined elsewhere where MS is barfing?

+1