Skip to content

Fix function page links breaking after first navigation#63

Merged
kyleamueller merged 1 commit into
mainfrom
fix/function-link-navigation
Apr 8, 2026
Merged

Fix function page links breaking after first navigation#63
kyleamueller merged 1 commit into
mainfrom
fix/function-link-navigation

Conversation

@kyleamueller
Copy link
Copy Markdown
Owner

Summary

  • Adds key={slug} to the <article> element on function pages so React fully re-mounts content when navigating between functions — fixes links in code blocks not working after the first click-through
  • Extracts parseInlineMarkdown into a shared utility (src/lib/markdown.ts) that auto-links M function names in backticks (e.g. Table.FromRecords becomes a clickable link to /functions/table-fromrecords)
  • Applies inline markdown parsing to example descriptions and step descriptions in ExampleSection, so function name references are clickable everywhere — not just in syntax-highlighted code blocks
  • Consolidates three duplicate parseInlineMarkdown implementations (functions, concepts, patterns pages) into one shared import

Test plan

  • npm run typecheck passes
  • npm test — 45/45 tests pass
  • Navigate Web.Contents → Json.Document → Table.FromRecords — all three clicks should work
  • Verify function names in Remarks text (e.g. Record.Field on Json.Document) are auto-linked
  • Verify step descriptions with function names render as clickable links
  • Check concept and pattern pages still render correctly after shared import change

🤖 Generated with Claude Code

Add key={slug} to the article element so React fully re-mounts page
content when navigating between function pages. Extract
parseInlineMarkdown into a shared utility (src/lib/markdown.ts) that
auto-links M function names in backticks to their function pages.
Apply inline markdown parsing to example descriptions and step
descriptions so function references are clickable everywhere.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@vercel
Copy link
Copy Markdown
Contributor

vercel Bot commented Apr 8, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
pqm-guide Ready Ready Preview, Comment Apr 8, 2026 0:18am

@kyleamueller kyleamueller merged commit 3cad34c into main Apr 8, 2026
6 checks passed
@kyleamueller kyleamueller deleted the fix/function-link-navigation branch April 8, 2026 00:22
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