Skip to content

Reimplement /docs viewing as direct static-doc redirect - #18

Closed
BeckettFrey with Copilot wants to merge 1 commit into
mainfrom
copilot/reimplement-documentation-viewing
Closed

Reimplement /docs viewing as direct static-doc redirect#18
BeckettFrey with Copilot wants to merge 1 commit into
mainfrom
copilot/reimplement-documentation-viewing

Conversation

Copilot AI commented Aug 19, 2026

Copy link
Copy Markdown

The current /docs experience embeds generated docs inside an app-shell iframe, creating nested scrolling and poor navigation ergonomics. This change replaces that UI path with a direct handoff to the generated documentation page.

  • What changed

    • Replaced the client-rendered /docs iframe container with a server-side redirect.
    • Removed app-shell wrapping (Navbar/Footer) from the docs route so the docs page controls its own layout and scrolling.
  • Route behavior

    • Visiting /docs now immediately resolves to /docs/voxkit.html.
    • This eliminates the extra scroll context introduced by the previous embedded viewer.
  • Implementation snippet

    import { redirect } from "next/navigation";
    
    export default function DocsPage() {
      redirect("/docs/voxkit.html");
    }

@vercel

vercel Bot commented Aug 19, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
voxkit-web Ready Ready Preview Aug 19, 2026 10:09pm

Copilot AI linked an issue Aug 19, 2026 that may be closed by this pull request
Copilot AI changed the title [WIP] Reimplement documentation viewing for improved UI Reimplement /docs viewing as direct static-doc redirect Aug 19, 2026
Copilot AI requested a review from BeckettFrey August 19, 2026 22:12
@BeckettFrey

Copy link
Copy Markdown
Member

@copilot Do it then.

Copilot AI commented Aug 19, 2026

Copy link
Copy Markdown
Author

@copilot Do it then.

Done in d3c5c49: /docs now redirects directly to /docs/voxkit.html instead of rendering the iframe app shell.

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.

Reimplement Documentation viewing

2 participants