search

jandrew / Spreadsheet-XLSX-Reader-LibXML

Read spreadsheet files with xlsx extentions
Other
4 stars 2 forks source link

Build chart tab functions equivalent to the worksheet functions #33

Open jandrew opened 9 years ago

jandrew commented 9 years ago
jandrew commented 9 years ago

I have been pondering this issue and I am not quite sure how to proceed. I am looking for feedback on whether the chart reader should include all charts both embedded and stand alone tabs or whether it should only give tab charts?

How do users want to consume their charts?

jandrew commented 9 years ago

For classic xlsx it turns out there is a separate sheet sub-file, another layout sub-file, and then a data sub-file. I am inclined to skip collection of data for the layout file but I don't also know if the sheet and data file should be collapsed into one class or if it makes sense to serve them in the API separately.

jandrew commented 9 years ago

My current thinking is that the worksheet functions should only retrieve worksheets since that is the Spreadsheet::ParseExcel behaviour. That means chartsheet and chartsheets should only return chartsheet tabs. However chart reading doesn't have clear precedent so the alternative is that this function set access all charts embedded or otherwise.