Skip to content

Remove the per-page Copy Markdown and Open buttons - #2

Merged
kyleconroy merged 1 commit into
mainfrom
remove-page-actions
Aug 27, 2026
Merged

Remove the per-page Copy Markdown and Open buttons#2
kyleconroy merged 1 commit into
mainfrom
remove-page-actions

Conversation

@kyleconroy

Copy link
Copy Markdown
Contributor

Removes the button row above each page body: fumadocs' MarkdownCopyButton ("Copy Markdown") and ViewOptionsPopover (the "Open" popover — view as markdown / edit on GitHub).

Also drops what only existed to support them:

  • the markdownUrl and sourcePath locals, and the getPageMarkdownUrl / gitConfig imports in app/(docs)/[[...slug]]/page.tsx
  • DocsDescription's mb-0 override, which was there so the description butted against the button row's bottom border — it falls back to the default mb-8

Left in place:

  • the /llms.mdx/docs/**/content.md routes — the copy button fetched them, but llms.txt and llms-full.txt still do
  • gitConfig, still used by lib/layout.shared.tsx for the navbar GitHub link

npm run types:check and npm run build pass; verified against a local static preview that neither control renders.

🤖 Generated with Claude Code

https://claude.ai/code/session_01GKc3BJAnDEG26khsB5p125

The button row above each page body held fumadocs' MarkdownCopyButton and
ViewOptionsPopover (the "Open" popover: view as markdown / edit on GitHub).
Drop both, along with the now-unused markdownUrl and sourcePath locals and
the getPageMarkdownUrl and gitConfig imports.

DocsDescription loses its mb-0 override and falls back to the default mb-8;
the override only existed so the description butted against the button
row's bottom border, which is gone with it.

The /llms.mdx/docs/**/content.md routes stay — the copy button fetched
them, but llms.txt and llms-full.txt still do. gitConfig stays in
lib/layout.shared.tsx for the navbar's GitHub link.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01GKc3BJAnDEG26khsB5p125
@kyleconroy
kyleconroy merged commit 0013c96 into main Aug 27, 2026
1 check passed
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