Add simple merge cells capability

[mchapman]

I needed this for a hack at the weekend. Didn't have time to do any tests, but it may save someone a few minutes.

[SheetJSDev]

@mchapman thanks :) A similar request was made for xlsx: SheetJS/sheetjs#41

I actually have a test file to play with: https://github.com/SheetJS/test_files/blob/master/merge_cells.xls

How are the other cells in a merge represented? Using the aforementioned file, how are cells B1, A2, B2, C2, D2, B3, and B4 stored?

[mchapman]

I didn't do anything in js-xls apart from return the array of merged cells,
and left what to do with them up to the calling app. In my case (
https://github.com/mchapman/getTriples/blob/master/server.js line 46) I
copied the contents of the top left cell into the other cells, but if you
were trying to represent the data in a spreadsheet-like style this would be
completely inappropriate.

On 10 March 2014 12:35, SheetJSDev [email protected] wrote:

@mchapman https://github.com/mchapman thanks :) A similar request was
made for xlsx: SheetJS/sheetjs#41SheetJS/sheetjs#41

I actually have a test file to play with:
https://github.com/SheetJS/test_files/blob/master/merge_cells.xls

How are the other cells in a merge represented? Using the aforementioned
file, how are cells B1, A2, B2, C2, D2, B3, and B4 stored?

Reply to this email directly or view it on GitHubhttps://github.com/SheetJS/sheetjs/pull/37#issuecomment-37177396
.

[SheetJSDev]

This looks OK, but I think it should save as range structures. See parse_Ref8U (which appears to be the same structure as Ref8) for what the range object looks like.

That way, your code simplifies greatly: for example, https://github.com/mchapman/getTriples/blob/master/server.js#L50 becomes XLS.utils.encode_cell(range.s)

[SheetJSDev]

@mchapman just saw the update. Looks good. If you can squash the two commits to one, I'll merge :)

[mchapman]

This was getting messy, so have started from scratch in #38