Skip to content

Fix lettered sub-lists rendering as code blocks in Policies 3.3F, 3.5, and 3.6 - #77

Open
amfriesz wants to merge 4 commits into
developmentfrom
60-fix-lettered-sublists-codeblocks
Open

Fix lettered sub-lists rendering as code blocks in Policies 3.3F, 3.5, and 3.6#77
amfriesz wants to merge 4 commits into
developmentfrom
60-fix-lettered-sublists-codeblocks

Conversation

@amfriesz

Copy link
Copy Markdown
Contributor

Summary

  • Policy 3.3F: Fix list numbering (a./b./c. sub-items under "Required Elements" were one column short of the parent numbered item's content indent, causing Pandoc to swallow them into a code block). Also fixes minor spacing/punctuation slips (missing space after a comma, double-spaces after periods, a merged word, a stray mid-sentence capital, inconsistent blank lines).
  • Policy 3.5: Fix the same class of bug in sections A, B, C, D, E, and H -- lettered/roman-numeral sub-items sat at a bare 4-space indent with no enclosing list, which is Markdown's classic indented-code-block trigger. Also fixes the **Guidelines** text under section H, which was rendering as literal asterisks because it was trapped in the same code block.
  • Policy 3.6: Fix the same bug in sections A (top), B, C, and D. The embedded "Investment Policy" body in the middle of section A already used well-formed bullet lists and was left untouched. Also converts the allocation-range disclaimer into a callout-note box, matching the pattern already used in 1-0-definitions.qmd and 3-5a-travel-expense.qmd.
  • All three files: prose paragraphs and list items rewrapped to ~80 columns (reflowing full paragraphs, not just individually-overflowing lines), matching the convention from commit 06815ad.

Fixes #60

Test plan

  • Rendered all three pages with Quarto and confirmed zero <pre>/<code> blocks remain
  • Confirmed nested lists render correctly (e.g. <ol type="a"> containing <ol type="1">)
  • Confirmed the **Guidelines** text in 3.5 now renders bold instead of literal asterisks
  • Confirmed the 3.6 callout renders as a titled note box and the allocation table is intact
  • Verified content is unchanged (whitespace-normalized diff) across all three files
  • Visual check of the rendered Quarto book once built

🤖 Generated with Claude Code

amfriesz and others added 3 commits August 26, 2026 15:15
- Convert the a./b./c. lettered sub-items under "Required Elements" so
  Pandoc no longer swallows them into an indented code block (they
  were one column short of the parent numbered item's content indent).
- Wrap all prose paragraphs and list items to ~80 columns, reflowing
  full paragraphs rather than only lines that individually overflowed.
- Remove a leftover leading-space indentation artifact on the intro
  heading/paragraph and the "Required Elements" label.
- Fix minor punctuation/spacing slips: a missing space after a comma,
  double-spaces after periods, a missing space ("Acoordinated" ->
  "A coordinated"), a stray mid-sentence capital, and a double blank
  line before the closing paragraph.

Verified by rendering: zero code blocks remain, content unchanged
aside from the fixes above.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
- Dedent the a./b./c./i./ii. sub-items across sections A-E and H from a
  bare 4-space indent to column 0. With no enclosing list, that indent
  was Markdown's classic "indented code block" trigger, so the
  sub-items rendered as literal preformatted text. This also fixes the
  "**Guidelines**" text under section H, which was rendering as a
  literal double-asterisk string because it was trapped in the same
  code block.
- Wrap all prose paragraphs and list items to ~80 columns, reflowing
  full paragraphs rather than only lines that individually overflowed.

Verified by rendering: zero code blocks remain, nested lists (e.g.
<ol type="a"> containing <ol type="1">) render correctly, and content
is unchanged aside from the fixes above.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
- Dedent the a./b./i./ii./iii./iv. sub-items in sections A (top), B, C,
  and D from a bare 4-space indent to column 0, fixing the same
  "indented code block" bug as Policy 3.5 -- the embedded Investment
  Policy body's existing "-   " bullet lists were already well-formed
  and left untouched.
- Convert the "[Note: ...]" allocation-range disclaimer into a Quarto
  callout-note box, matching the pattern already used in
  1-0-definitions.qmd and 3-5a-travel-expense.qmd.
- Wrap all prose paragraphs and list items to ~80 columns, reflowing
  full paragraphs rather than only lines that individually overflowed.

Verified by rendering: zero code blocks remain, nested lists render
correctly, the callout renders as a titled note box, the allocation
table is intact, and content is unchanged aside from the fixes above.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@amfriesz
amfriesz requested a review from BenGalewsky August 26, 2026 19:18
@BenGalewsky

Copy link
Copy Markdown
Collaborator

@amfriesz - I made some more formatting changes to this PR, but otherwise ready to approve

@BenGalewsky BenGalewsky left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Looks good with some additional changes pushed

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