Repository navigation
Conversation
Clicking an addon row in the addon manager or the registry dialog opens a detail view with the addon's long description, a screenshot carousel, author/version/license info, a repository link, and the install/update/unlock action. Screenshots are fetched over plain HTTPS from URLs published by the registry (pinned to the addon's release tag) and cached on disk, so no git installation is required on any platform. Addons without registry data still show their local metadata.
Reuse the existing MarkdownView widget (also used for machine notes) instead of a plain label, so addon authors can use headings, lists, emphasis, links, and code blocks in long_description.
Give every builtin addon a Markdown long_description and screenshots in its manifest, so the detail view is populated out of the box. Local manifests list repo-relative screenshot paths; the detail view resolves them against the addon's repository (the Rayforge repo for builtins) at the version tag, falling back to the default branch for development builds. AsyncImage gains a fallback URL for this. Add screenshot targets for the two addons that had no screenshots: app-settings:tools (tool library page) and addon:cnc:adaptive-clearing (CNC step settings on a spindle machine).
Update the translated addon and addon-manifest docs pages in all
seven website languages for the detail view, Markdown descriptions,
and screenshots.
Extract the new 'by {author}' UI string and translate it in every
shipped locale; en keeps the empty fallback.
Replace the CNC settings-dialog screenshot with the actual adaptive clearing toolpath rendered in the 3D canvas: a pocket with two islands, cleared on a spindle machine, at 80% playback. Add a dedicated pocket SVG asset for the target.
This branch was successfully deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Clicking an addon row — in the installed addons list or in the Addon Registry dialog — now opens a detail view with:
MarkdownViewwidget),action_buttonhelper.Companion registry PR (merged): barebaric/rayforge-registry#13.
Builtin addons
All 10 builtin addons now ship a Markdown
long_descriptionandscreenshotsin their manifests, so the detail view is populated out of the box. Local manifests list repo-relative screenshot paths; the detail view resolves them against the addon's repository (the Rayforge repo for builtins) at the version tag, falling back to the default branch for development builds.AsyncImagegained a fallback URL for this.Two new screenshot targets cover the addons that had no screenshots:
app-settings:tools— the tool library settings page (seeds sample tools)addon:cnc:adaptive-clearing— the actual adaptive clearing toolpath in the 3D canvas (a pocket with two islands, cleared on a spindle machine, at 80% playback)The detail-view screenshot (
app-settings:addons:detail) was regenerated and now shows the carousel and Markdown rendering.Implementation notes
AddonMetadatagainslong_descriptionandscreenshots. Registry entries keep only absolute HTTP(S) URLs; local manifests keep sanitized repo-relative paths.CACHE_DIR/addon-images, sha256-keyed, 5 MB cap) and decoded with pyvips (PNG/JPEG/WebP, shrink-only). No git is required on any platform.AddonManager.fetch_registry_cached()(5 min TTL) avoids re-downloading the registry for every detail view; failed fetches are not cached.Translations
by {author}UI string is extracted and translated in every shipped locale; catalogs regenerated viaupdate-translationsand verified additions-only withmsgfmt --check.Testing
tests/addon_mgr: 161 passed (parsing, registry cache, local screenshot paths).xvfb-run pytest -m ui): 40 passed (dialog content, Markdown rendering, screenshot resolution incl. tag/main/non-GitHub cases, fallback download, row activation in both lists — all offline).pixi run lint(ruff, flake8, pyflakes, pyright): clean; also ran as the pre-commit hook on every commit.