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

Fix deeply nested prose breaking pdflatex #147

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Conversation

jaehyun1ee
Copy link

Fixes #146, this fixes the issue around deeply nested prose breaking the PDF backend.

Briefly, PDF is generated by: spliced rST -- Sphinx --> TEX -- pdflatex --> PDF, and previously > 5 levels of nesting of ordered lists broke pdflatex. This PR adds LaTeX enumitem package to the preamble of Sphinx configuration to allow more levels of prose nesting.

This is not the best solution to fix the problem, e.g., we may optimize the generated prose to remove redundant nestings. Yet this is a quick alternative for the time being.

@jaehyun1ee jaehyun1ee requested a review from rossberg February 6, 2025 00:31
Copy link
Collaborator

@rossberg rossberg left a comment

Choose a reason for hiding this comment

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

LGTM

@@ -298,7 +298,18 @@

# Additional stuff for the LaTeX preamble.
# Don't type-set cross references with emphasis.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Can you update this comment?

Copy link
Author

Choose a reason for hiding this comment

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

I've updated the comment, mentioning the enumitem package.

@jaehyun1ee
Copy link
Author

@rossberg can we get this merged?

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.

(**P1**) Todo about the inability to render the rule for return_call_ref & handler due to deep nesting.
2 participants