Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Top-level column headers are missing in XLS exports #7486

Closed
AleksueiR opened this issue Dec 4, 2017 · 3 comments
Closed

Top-level column headers are missing in XLS exports #7486

AleksueiR opened this issue Dec 4, 2017 · 3 comments
Assignees

Comments

@AleksueiR
Copy link

@AleksueiR AleksueiR commented Dec 4, 2017

Expected behaviour

For XLS exports, I expect the table to have two-level headers since the accessibility module adds top-level columns:
HTML table:

XLS:

Actual behaviour

The top-level headers are missing in the actual output.

Other

Same happens with CSV exports, but I don't think having two-level headers is a good idea for CSV files. They should probably keep the headers as they rendered without the accessibility module:

Live demo with steps to reproduce

http://jsfiddle.net/ouroborosDragon/aga1k7nd/10/

Affected browser(s)

All.

@KacperMadej
Copy link
Contributor

@KacperMadej KacperMadej commented Dec 5, 2017

Hi,

Thank you for reporting.
This looks more like an enhancement suggestion than a bug, because both modules are not directly connected. The info is there, but given in a different format.

@AleksueiR
Copy link
Author

@AleksueiR AleksueiR commented Dec 5, 2017

Not all info is there, when the accessibility module is added, the series names are stripped from the export:

without accessibility module

http://jsfiddle.net/ouroborosDragon/aga1k7nd/10/

with accessibility module

http://jsfiddle.net/ouroborosDragon/aga1k7nd/11/

@KacperMadej KacperMadej changed the title Tol-level column headers are missing in XLS exports Top-level column headers are missing in XLS exports Dec 6, 2017
@TorsteinHonsi
Copy link
Collaborator

@TorsteinHonsi TorsteinHonsi commented Dec 11, 2017

We're currently working on this, expect a fix very soon.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
4 participants