Skip to content

Fix double introductions - #1059

Open
siwelwerd wants to merge 3 commits into
mainfrom
100426-fix-double-introductions
Open

siwelwerd wants to merge 3 commits into
mainfrom
100426-fix-double-introductions

Conversation

@siwelwerd

Copy link
Copy Markdown
Contributor

Addresses #1056 , following @StevenClontz's suggestion to nest everything in double introductions.

I think HTML looks okay, hence this draft PR. But the print PDF is not building:

LaTeX Warning: Reference `LE' on page 260 undefined on input line 12518.

Package hyperref Warning: The anchor of a bookmark and its parent's must not
(hyperref)                be the same. Added a new anchor on input line 12518.

LaTeX Warning: Reference `LE1' on page 260 undefined on input line 12519.

! Undefined control sequence.
\kvtcb@before@upper ...\tcbtitle \fi \setparstyle 
                                                  
l.12519 ...}{{\xreffont\ref*{LE1}}.}{}{sample-LE1}

I'll have to come back to chase this one down, I think I added in some xrefs from the sample problems to the relevant sections so the print reader would see a page number reference.

@github-actions

github-actions Bot commented Oct 5, 2026

Copy link
Copy Markdown

🚀 Preview available 🚀

https://3297ba05.tbil.pages.dev

@github-actions

github-actions Bot commented Oct 7, 2026

Copy link
Copy Markdown

🚀 Preview available 🚀

https://4f5497c6.tbil.pages.dev

Comment thread xsl/print.xsl
<xsl:param name="latex.preamble.late">
<xsl:text>% Renew tcolorbox for exercise&#xa;</xsl:text>
<xsl:text>\tcbset{ divisionexercisestyle/.style={bwminimalstyle, runintitlestyle, exercisespacingstyle, before skip={3 ex}, breakable, before upper app={\setparstyle} } }&#xa;</xsl:text>
<xsl:text>\tcbset{ divisionexercisestyle/.style={bwminimalstyle, runintitlestyle, exercisespacingstyle, before skip={3 ex}, breakable, before upper app={\ptxsetparstyle} } }&#xa;</xsl:text>

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

This was annoying to track down. Apparently PreTeXt quietly changed the name of this command (probably for a very good reason) and so it was breaking the build of the print version that used this custom xsl.

@github-actions

github-actions Bot commented Oct 8, 2026

Copy link
Copy Markdown

🚀 Preview available 🚀

https://9e112a18.tbil.pages.dev

@siwelwerd
siwelwerd requested a review from StevenClontz October 8, 2026 13:29

This branch was successfully deployed

1 active deployment
preview — 9be2f843 Deployed Oct 8, 2026 by github-actions[bot]
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.

2 participants