Skip to content

Render italics in glossary related terms on main - #1111

Open
LukasWallrich wants to merge 1 commit into
mainfrom
glossary-related-terms-italics
Open

LukasWallrich wants to merge 1 commit into
mainfrom
glossary-related-terms-italics

Conversation

@LukasWallrich

@LukasWallrich LukasWallrich commented Nov 11, 2026 •

Copy link
Copy Markdown
Contributor

Ports the related-terms change from #1085 (merged into rebrand-doors) to main, so #1092 can merge before the Doors launch. The three lines in layouts/glossary/single.html are identical to #1085's, so merging main into rebrand-doors later should not conflict.

What changes

  • Related terms strip * and \ before building the link slug, so *p*-hacking links to /glossary/english/p_hacking/.
  • Link text and unlinked terms are rendered with markdownify, so *p* shows as an italic p.

Effect on current content (all 1,035 related-terms lines compared before and after on main)

  • 12 lines change, all for the better: & is now escaped as &; a trailing space inside the "Meta-science or Meta-research" link is gone; the stray "Incorrect definition:" text on the Statistical significance pages renders bold instead of showing asterisks ([HOLD] 📘 Glossary update - 2026-11-09 (replaces #1084) #1092 removes that text).

With #1092's glossary content

  • p-hacking (35 links) and p-value (20 links) render italic and link to their pages. p-value fishing and Reverse p-hacking render italic without a link, as they have no glossary page.
  • hugo builds and scripts/check_seo_html.py passes on both builds.

Not included: italic rendering of term titles (e.g. "Research Contribution Metric (*p*)" in the sidebar). That part of #1085 depends on the Doors title partial.

🤖 Generated with Claude Code


Summary by cubic

Ports the glossary related-terms fix from rebrand-doors to main so related terms render Markdown correctly before the Doors launch.

  • Related terms like *p*-hacking now link to /glossary/english/p_hacking/ and show an italic p.
  • All 1,035 related-terms lines were compared before and after: 12 change, fixing & escaping, a trailing space, and bold rendering of "Incorrect definition:".
  • Hugo builds and scripts/check_seo_html.py pass on both builds.

Written for commit 8fa5fa7. Summary will update on new commits.

View guided diff Turn on auto-fix

Ports the related-terms part of #1085 from rebrand-doors to main, so
*p*-hacking and *p*-value show an italic p and link to their pages.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@LukasWallrich
LukasWallrich requested a review from a team as a code owner November 11, 2026 17:00
@github-actions

Copy link
Copy Markdown
Contributor

👍 All image files/references (if any) are in webp format, in line with our policy.

@cubic-dev-ai cubic-dev-ai Bot 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.

No issues found across 1 file

View guided diff | Turn on auto-fix | Re-trigger cubic

@github-actions

Copy link
Copy Markdown
Contributor

📝 Spell Check Results

Found 1 potential spelling issue(s) when checking 1 changed file(s):

📄 layouts/glossary/single.html

Line Issue
32 Referenz ==> Reference

ℹ️ How to address these issues:

  1. Fix the typo: If it's a genuine typo, please correct it.
  2. Add to whitelist: If it's a valid word (e.g., a name, technical term), add it to .codespell-ignore.txt
  3. False positive: If this is a false positive, please report it in the PR comments.

🤖 This check was performed by codespell

@LukasWallrich

LukasWallrich commented Nov 11, 2026 •

Copy link
Copy Markdown
Contributor Author

✅ Staging Deployment Status

This PR has been successfully deployed to staging as part of an aggregated deployment.

Deployed at: 2026-11-13 12:30:41 UTC
Staging URL: https://staging.forrt.org

The staging site shows the combined state of all compatible open PRs.

This branch has not been deployed

No deployments
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.

1 participant