Repository navigation
docs: review pass on accuracy, clarity, and styling - #68
Merged
Merged
Conversation
- Migration: rename the old Komga container so `docker run --name=kmrs` no longer collides with it, and explain switching back - Privacy: credentials live in the app database, not Keychain; disclose model-pack downloads from models.kmworks.date and the opt-in comic metadata sent to translation providers - Platforms: no Windows aarch64 build, and binaries are self-contained rather than static - Installation: libpdfium is needed for PDFs with release binaries; komf auth wording matches the komf page; Windows tray as a list - Trackers: document disconnecting, library filter, default platform - Fix history retention key, sidebar orders, Komga capitalization - Group behavior differences under subheadings - FAQ/compare/landing: drop duplicate FAQ, balance the compare table, correct screenshot dimensions and alt text, hang Apple devices under KMReader only in the ecosystem diagram Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
- Header CTA uses the amber brand button and offers the KMServer download on server pages - Docs links and the current sidebar entry use the kmworks accent - Inline code chips keep their border when a long path wraps - Compare table keeps the capability column sticky on phones - Hero screenshots stay inside the page gutter on narrow screens Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
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
A review pass over the docs, landing pages, and styles. Claims were checked against the KMReader and kmrs sources.
Accuracy
docker run --name=komgawhile the stoppedkomgacontainer still existed, which fails with a name conflict. The guide now renames the old container tokomga-javaand names the new onekmrs, the same name the Compose examples use. It also explains how to switch back and thatkmrs.sqliteholds KMServer-only data.models.kmworks.date; the policy now discloses this.[history] retention-dayskey.ordercollisions resolved.Clarity
width/heightmatch the real files (the iPad shots are 4:3 landscape), which avoids layout shift.Styling
/server/*pages.Test plan
pnpm typecheck,pnpm build,pnpm lint:docs(internal links included),nimbus-docs check: all clean🤖 Generated with Claude Code