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

LaTeX: allow again figure inside seealso, and seealso inside table cell #12598

Merged
merged 4 commits into from
Jul 16, 2024

Conversation

jfbu
Copy link
Contributor

@jfbu jfbu commented Jul 16, 2024

This fixes #12594 which emerged because #12508 had modified seealso and note-like admonitions to use background color and other effects, but the ensuing LaTeX can not work in a table cell without extras.

This also fixes unreported issues unrelated to the 7.4.0 release:

  • (now old style "lightbox") admonitions render badly in tabulary,
  • "heavybox" admonitions (they are all now but warning et al. were already) cause PDF crash if in tabulary and rendered badly if in tabular or longtable.
  • figure in a table cell is not properly separated from immediately preceding text.

This fixes sphinx-doc#12594 which emerged because sphinx-doc#12508 had modified seealso and
note-like admonitions to use background color and other effects, but the
ensuing LaTeX can not work in a table cell without extras.

This also fixes unreported issues unrelated to the 7.4.0 release:
- (now old style "lightbox") admonitions render badly in tabulary,
- "heavybox" admonitions (they are all now but warning et al. were
  already) cause PDF crash if in tabulary and are rendered badly if in
  tabular or longtable.
- figure in a table cell is not properly separated from immediately
  preceding text.
@jfbu
Copy link
Contributor Author

jfbu commented Jul 16, 2024

I will push soon a commit to fix failing test (which fails for rather silly reason that some label stepped by 1 unit due to additional content; but I should have tested beforehand at home).

@AA-Turner AA-Turner merged commit 152b783 into sphinx-doc:master Jul 16, 2024
21 checks passed
@chrisjsewell
Copy link
Member

Cheers @jfbu

@jfbu jfbu deleted the latex_12594 branch July 17, 2024 08:48
@AA-Turner AA-Turner added this to the 7.4.x milestone Jul 17, 2024
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 17, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

LaTeX regression for admonitions
3 participants