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

Table formatting bug #36

Closed
itskun opened this issue Feb 9, 2017 · 1 comment
Closed

Table formatting bug #36

itskun opened this issue Feb 9, 2017 · 1 comment
Assignees

Comments

@itskun
Copy link

itskun commented Feb 9, 2017

Given a table with 7 cols and 2 rows:
image
When I try to bring a table to a form, the value in cell B2 is lost:
image
Now, when I try to connect F1 and F2 rowspan getting corrupted pptx file. Code:
var slide2 = pptx.addNewSlide(); var rows2 = [ [ {text:'A0 and B0', opts:{rowspan:2}},{text:'A1 and A2', opts:{rowspan:2}},{text:'B1 and C1', opts:{colspan:2}},{text:'D1 and E1',opts:{colspan:2}},'F1' ], [ 'B2','C2','D2','E2','F2' ] ]; var tabOpts2 = { x: 0.5, y: 1.5, w: 13, margin:0 }; var celOpts2 = { font_size: 10, font_face: 'Times New Roman', align: 'center', valign: 'middle', border: { pt: '1' } }; slide2.addTable( rows2, tabOpts2, celOpts2 );

@gitbrent gitbrent self-assigned this Feb 9, 2017
gitbrent pushed a commit that referenced this issue Feb 16, 2017
Resolves Issue #36 including complete re-write of table building code.
@gitbrent
Copy link
Owner

Hi @itskun ,

Thanks very much for reporting this!

Apparently, the test cases i had in the table demo pages were the only ones that worked... Sequential rowspans and/or colspans were causing issues across the board and necessitated a complete re-write of the table building algorithm.

I've created some complex test cases and added them to the demo page.

screen shot 2017-02-15 at 18 13 18

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

No branches or pull requests

2 participants