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

add support for \LTcaptype, issue #1377 #1398

Merged
merged 1 commit into from
Jul 6, 2024
Merged

Conversation

u-fischer
Copy link
Member

Status of pull request

  • Ready to merge

Checklist of required changes before merge will be approved

  • Test file(s) added
  • Version and date string updated in changed source files
  • Relevant \changes entries in source included
  • Relevant changes.txt updated
  • Rollback provided (if necessary)? Imho not needed
  • ltnewsX.tex (and/or latexchanges.tex) updated

Comment on lines +934 to +935
% The name used as counter, in caption, "\addcontentsline" and in targets.
% "\providecommand" is used for compability with \pkg{ltcaption}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We typically use "|"

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@josephwright not my fault ;-)

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

longtable predates latex2e:-) it has % \DeleteShortVerb{\|} (it has several fragments that want a verbatim |)

@davidcarlisle davidcarlisle merged commit 7c96b6b into develop Jul 6, 2024
82 checks passed
@davidcarlisle davidcarlisle deleted the gh1377-longtable branch July 6, 2024 08:25
muzimuzhi added a commit to muzimuzhi/latex2e that referenced this pull request Jul 6, 2024
And reformat longtable.dtx.
davidcarlisle pushed a commit that referenced this pull request Jul 8, 2024
* Correct typos introduced in #1401

* Correct typos introduced in #1398

And reformat longtable.dtx.
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