Skip to content

Conversation

christopherkenny
Copy link
Contributor

@christopherkenny christopherkenny commented Sep 27, 2025

I believe this is just on the scale of a typo, so I haven't filled out the checklist, but just let me know if you disagree and I'll update.

Description

quarto_super emits figure captions as paragraphs (par(...)) because they are not placed in block(...) calls.

This is generally not a problem, since they're both text and will typically show fine. However, this becomes a niche issue when doing things like numbering paragraphs as in ctk-legal, as the captions get captured as paragraphs. (Example PDF with the updated quarto_super using a defintions.typ partial.)

See this discussion in the Typst forum, where this was pointed out.

Checklist

I have (if applicable):

  • filed a contributor agreement.
  • [N/A] referenced the GitHub issue this PR closes
  • updated the appropriate changelog in the PR
  • ensured the present test suite passes
  • added new tests
  • [N/A] created a separate documentation PR in Quarto's website repo and linked it to this PR

@posit-snyk-bot
Copy link
Collaborator

posit-snyk-bot commented Sep 27, 2025

🎉 Snyk checks have passed. No issues have been found so far.

security/snyk check is complete. No issues have been found. (View Details)

license/snyk check is complete. No issues have been found. (View Details)

@cscheid
Copy link
Collaborator

cscheid commented Sep 29, 2025

I think this is a good fix (but it's not just a typo, because it does affect downstream code potentially).

Could you add an entry to the changelog? Thanks.

@cscheid
Copy link
Collaborator

cscheid commented Oct 1, 2025

Perfect. Thank you!

@cscheid cscheid merged commit 0eda022 into quarto-dev:main Oct 1, 2025
51 checks passed
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