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

Provides Longtable-to-xtab filter #154

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

jdutant
Copy link
Contributor

@jdutant jdutant commented Jan 17, 2021

Convert Pandoc's LaTeX table output from longtable to xtab. This addresses the issue of longtables not being compatible with multiple columns layouts.

Could be improved by turning table headers into proper xtab header code (see README).

@tarleb
Copy link
Member

tarleb commented Jan 18, 2021

Those TeX hacks are really clever; I never thought about changing tables by redefining the commands.
I haven't done a complete review yet, but looks good. I only noticed some inconsistent indentation in Lua and overlong lines in the README.

@jdutant
Copy link
Contributor Author

jdutant commented Jan 26, 2021

Thanks I'll clean that up asap.

@pbsds
Copy link

pbsds commented Feb 15, 2022

Seems like this table

| kjh | kjh |
| --- | --- |
| 1   | 2   |
| 4   | 3   |

: this is a test {#tbl:asd}

results in this error for me:

Error producing PDF.
! Missing \endgroup inserted.
<inserted text> 
                \endgroup 
l.142 \caption{\label{tbl:asd}this is a test}

The label is not the culprit.

pandoc 2.14.0.3
Compiled with pandoc-types 1.22.1, texmath 0.12.3.2, skylighting 0.10.5.2,
citeproc 0.4.0.1, ipynb 0.1.0.2

Update:

Adding this line did the trick for me:

element.caption.long = {}

I then got errors due to \fi, so i disabled the if #element.head[2] > 0 then statement.

I'll try to report here if i run into any issues with this solution.

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

Successfully merging this pull request may close these issues.

3 participants