FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Add an API reference page for the `preview` module · Issue #41 · pythonnative/pythonnative · GitHub

Repository navigation

Add an API reference page for the preview module #41

Description

Problem

Every core module has an API reference page under docs/api/ (hot_reload.md, diagnostics.md, screen.md, and so on), but src/pythonnative/preview.py has none. The desktop preview only has the guide at docs/guides/desktop-preview.md, so the public run_preview() (line 371) and DesktopApp (line 76) have no rendered reference docs.

Suggested fix

  • Add docs/api/preview.md using the same thin mkdocstrings pattern as an existing page (for example, docs/api/hot_reload.md): a short intro plus a ::: pythonnative.preview block.
  • Add the page to the API section of the nav in mkdocs.yml (around lines 105-135), keeping alphabetical order with its neighbors.
  • Link the new page from docs/guides/desktop-preview.md where it discusses run_preview.

Verification

Run mkdocs build --strict and confirm the page renders with mkdocs serve.

Suggested commit message

docs(mkdocs): add preview API reference page

Activity

andevsr commented on Sep 2, 2026

Hi, I would like to work on this issue. Could you please assign it to me?

Abhi-ai2975 commented on Sep 2, 2026

Hey, I am intrested in this can you assign it to me?

andevsr commented on Sep 6, 2026

Upstream's runtime overhaul (#74/#80) replaced the desktop preview and now documents pythonnative.preview on the Dev server API page, so this is obsolete.

Closing

owenthcarey commented on Sep 9, 2026

ContributorAuthor

Thanks @andevsr, you're right. #74 replaced the Tkinter desktop preview with the browser preview, and the current preview module is already rendered on the Dev server API page. The browser preview guide also links to the current entry point.

DesktopApp and run_preview() are gone; the current entry points are DevSession and serve(). Adding a separate page using the old instructions would document removed APIs and duplicate the existing module reference.

Closing as completed because the reference-documentation goal is already covered. Sorry I left the issue open after your comment, and thanks for checking the new architecture before doing unnecessary work.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

documentationImprovements or additions to documentationgood first issueGood for newcomers

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions


    Back | FazBrowse Home | New Git URL