Skip to content

1d: stop three paragraphs rendering as tables (bare | in inline math) - #254

Merged
teorth merged 1 commit into
teorth:mainfrom
JeetuSK0808:fix-1d-pipes
Oct 26, 2026
Merged

teorth merged 1 commit into
teorth:mainfrom
JeetuSK0808:fix-1d-pipes

Conversation

@JeetuSK0808

Copy link
Copy Markdown
Contributor

Three paragraphs on the 1d page currently render as one-row tables on the live site: line 11 ("This is the maximum of |f*f| ..."), and the "How the upper bound is checked" and "Other variant" bullets. They use |...| inside $...$, and kramdown reads the bare pipes as table cell separators before MathJax sees them.

This PR switches those to \lvert ... \rvert, as the Rendering section of CONTRIBUTING recommends. The mathematics is unchanged. Rendering 1d.md with kramdown (GFM input, mathjax engine, as Jekyll does) gives 5 tables before the change and 2 after (the two bound tables). There's no new emphasis, and the edit only touches lines that #247 doesn't.

Found with a small consistency checker I'm putting together for the repo. I'll open that separately in case it's useful.

Prepared with Claude Code (Anthropic).

🤖 Generated with Claude Code

Lines 11, 30 and 32 used |...| inside $...$. kramdown reads the bare pipes as table cells,
so each of those paragraphs rendered as a one-row table. Use \lvert ... \rvert as CONTRIBUTING
recommends. No change to the mathematics.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

@chessing234 chessing234 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

all good

@teorth
teorth merged commit a492867 into teorth:main Oct 26, 2026
teorth added a commit that referenced this pull request Oct 26, 2026
That branch predated #254, so merging it brought back bare pipes in inline
math on two lines, which kramdown renders as stray one-row tables. Caught
by check_pages.py W3 -- the second time the tool has paid for itself today.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
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