leftshifters / excel-parser

nodejs wrapper for parsing spreadsheet.
MIT License
72 stars 70 forks source link

Parse merged cells #13

Closed vlad-ivanov-name closed 8 years ago

vlad-ivanov-name commented 10 years ago

Will this feature be useful? I needed to parse XLS file with lots of merged cells. python-xlrd supports it, so I tried to modify this library to return merged cells too. I was thinking about converting WSheet.merged_cells to JSON and passing it from Python script to Node.js. However, this feature breaks the behaviour of how the code works, because merged cells can't be added to "parsed" array.

cb(null, parsed, mergedCells);

In your opinion, what is the proper way to add functionality like this? Moreover, python-xlrd also provides cell formating extraction and other nice features, but for now these functions will require a change of data representation in JS.