Repository navigation
🎨 Apply the brand skin to the docs - #699
Conversation
Brand colors from brand/palette.json, new logos, favicon, Open Graph image, and the rendered hero on the landing page. Remove the old logo, hero, and favicon files, which nothing references anymore. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
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. |
|
No actionable comments were generated in the recent review.
 |
| Layer / File(s) | Summary |
|---|---|
Embed picker demos docs/dialogs/directory-picker.mdx, docs/dialogs/gallery-picker.mdx |
Both pages import FileKitDemo and add a “Try it” section describing and embedding the corresponding browser demo. |
Documentation branding
| Layer / File(s) | Summary |
|---|---|
Theme and social metadata docs/docs.json |
Theme colors and favicon and logo paths change. Open Graph image metadata and Twitter large-image card metadata are added. |
Hero and navbar images docs/introduction.mdx, docs/style.css |
The introduction uses /brand/hero.webp with descriptive alt text, dimensions, full-width rounded styling, and noZoom. The navbar logo height is set to 1.375rem. |
Priority:
Estimated code review effort: 2 (Simple) | ~10 minutes
Change: Feature
Possibly related PRs
- vinceglb/FileKit#694: Adds the
FileKitDemocomponent and the demo identifiers used by the new embeds. - vinceglb/FileKit#698: Applies the brand palette and favicon to the web sample, which the PR summary notes is handled separately.
- vinceglb/FileKit#695: Adds the brand asset pipeline that generates the documentation images and icons used here.
Merge Risk: 
2dcc6
-
The documentation branding and picker demos are ready to merge with no supported material risk identified.
Architecture risk: 
2dcc6
-
The change affects 1 system.
Changed systems:
docsArchitecture concerns
No architecture-level concerns identified.Review details
-
Systems and components
- observed — docs (service) was modified; 5 changed files map to changed impact.
Before / after behavior
- observed — Modified behavior in docs/dialogs/directory-picker.mdx: Added the
FileKitDemoimport. - observed — Modified behavior in docs/dialogs/directory-picker.mdx: Added a “Try it” section describing the browser directory-picker demo and embedding it with the
directory-pickerdemo identifier. - observed — Modified behavior in docs/dialogs/gallery-picker.mdx: Adds the
FileKitDemoimport for embedding the sample. - observed — Modified behavior in docs/dialogs/gallery-picker.mdx: Adds a “Try it” section describing image, video, selection-mode, and metadata interactions in the browser, and embeds the gallery-picker demo.
-
Pre-merge checks: 
-
Check name Status Explanation Title check The title clearly and concisely describes the primary change: applying the new brand styling to the documentation site. Description check The description explains the changes, affected scope, validation steps, and follow-up items. It does not include the template's checklist or issue-closing reference, but the main required information … Docstring Coverage No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentat… Linked Issues check Check skipped because no linked issues were found for this pull request. Out of Scope Changes check Check skipped because no linked issues were found for this pull request. Full details: Description check
-
Explanation
The description explains the changes, affected scope, validation steps, and follow-up items. It does not include the template's checklist or issue-closing reference, but the main required information is present.
Full details: Docstring Coverage
-
Explanation
No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentation-only edits and deletions within surviving functions are included; unchanged diff context and fully deleted functions are excluded. Analyzed 0 functions across 0 files. (5 skipped: 5 unsupported.)
-
Finishing Touches
-
🧪 Generate unit tests (beta)
-
- Commit to this branch
- Create a new PR
-
- Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
Comment @coderabbitai help to get the list of available commands.
What
Applies the new brand to the docs site and adds the live demo to two more pages.
docs/docs.json: brand colors frombrand/palette.json(primary#1A7F4B, light#E3C266, dark#1A7F4B), the new light and dark logos, the brand favicon, and a site-wide Open Graph image (/brand/og.jpg, large Twitter card).docs/introduction.mdx: the landing page uses the rendered hero (/brand/hero.webp). It drops theFramebecause the image already has rounded corners.docs/style.css: makes the navbar logo smaller (22px instead of 28px). The brand logo is cropped tight to its ink, so it looked too large at the default height.docs/logo/*,docs/images/hero-*,docs/favicon.svg). Nothing in the repo references them anymore.Platforms
Docs only. No library or sample changes.
Validation
mint dev: checked the landing page, File picker, Gallery picker, and Directory picker in light and dark mode. Clicking "Run live demo" loads the deployed sample on the right screen with the matching theme.mint broken-links: no broken links.Follow-up
https://filekit.mintlify.appURL. Update it if the docs move to a custom domain.🤖 Generated with Claude Code
Summary by CodeRabbit