Repository navigation
docs: fix hosting badges, installation link and brand typos - #496
Conversation
Preview Deployment
Built from 0fd5a4d |
Strix Security ReviewWarning This pull request has 1 commit after the last Strix review ( No security issues found. Review summaryReviewed all seven changed files in PR #496: three documentation pages (brand guidelines, FAQ, installation index, and the managed-hosting page) and three new static SVG hosting badges. The changes are documentation and styling only — prose rewrites, markdown link/anchor updates, and self-contained SVG image assets served via Updated for Reviewed by Strix |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 00b6ff7aaa
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| <a href="https://www.pikapods.com/pods?run=navidrome"><img style="height:40px;width:auto" src="/images/hosting/pikapods.svg" alt="Run on PikaPods"></a> | ||
|
|
||
| ### Zenith | ||
| ### Zenith Hosting |
There was a problem hiding this comment.
Preserve the published Zenith anchor
Renaming this heading changes its generated fragment from #zenith to #zenith-hosting. The previous FAQ and installation pages both published links to /docs/installation/managed/#zenith, so bookmarked or externally copied versions of those URLs will now land at the top of the page instead of the provider section. Keep an explicit zenith anchor (or equivalent compatibility target) while displaying the longer heading.
Useful? React with 👍 / 👎.
|
|
||
| Keep it true. If your app only supports the Subsonic API, don't say it supports Navidrome-only | ||
| features. | ||
| **App and project names.** Your app needs its own name. You can add "Navidrome" at the end of the name to tell people what it works with. |
There was a problem hiding this comment.
Align the naming rule with the allowed examples
This sentence now says that Navidrome must be at the end of the app name, but the same page explicitly approves MyAmazingApp - Navidrome & Jellyfin, where it is not last. Developers relying on this brand-permission page therefore cannot determine whether multi-service suffixes are permitted; retain the earlier “after your own app name” wording or update the examples to match the stricter rule.
Useful? React with 👍 / 👎.
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
Description
The managed-hosting changes in #485 render Zenith at 49px high beside a 40px PikaPods button, and ElfHosted is a plain link. Carry forward that contributor's commits onto current master and render all three providers as matching 200×40px deployment badges.
The SVGs live in
static/images/hosting/and match the README assets. PikaPods and Zenith preserve their original artwork, scaled uniformly on centered, padded canvases with matching brand-color backgrounds. ElfHosted uses its original mascot in a self-contained deployment badge. Logo proportions, provider links and hosting text are retained.Also fix the broken relative ElfHosted link on the installation page and clear brand-page typos (
at the of→at the end of,som→some,explicitely→explicitly,eg.→e.g.). The original brand guidance and hosting claims are otherwise preserved.Related to #485. Also reviewed #469, which constrains Zenith's height but does not cover ElfHosted or equal-width badges. Existing contributor PRs remain open and untouched.
Type of change
Checklist
git diff --checkand asset checks passValidation
All three badges load and render exactly 200×40px with proportional artwork. Every changed website file matches the source used for the local build and screenshots. SVG path checks confirm original provider artwork is unchanged; assets are self-contained and identical across both repositories. The installation-page link resolves to
/docs/installation/managed/#elfhosted.The full build used no user data or real secrets. Translation statistics were omitted from local validation; unrelated public app-update requests produced expected unauthenticated API warnings.
The
[skip ci]commit intentionally prevents Preview Build from triggering automatic Cloudflare deployment, honoring the request to avoid website deployment. CI/deployment checks are therefore skipped. No website was deployed.Full page screenshots and labeled side-by-side comparisons of real badge crops are saved in Library. Existing mobile page overflow was observed before and after; the three badges fit within the content column.
Summary by CodeRabbit