Skip to content

Update favicon to square brand icon - #1234

Merged
comfyui-wiki merged 1 commit into
mainfrom
fix/square-favicon
Jul 7, 2026
Merged

Update favicon to square brand icon#1234
comfyui-wiki merged 1 commit into
mainfrom
fix/square-favicon

Conversation

@mobeenabdullah

Copy link
Copy Markdown
Contributor

Point the docs favicon at the square, full-bleed brand mark (ink background, yellow C), matching the org-wide standardization. A square source lets each platform apply its own corner mask instead of double-rounding a pre-rounded asset.

Changes:

  • Add favicon.svg (the square ink mark).
  • Update docs.json favicon from /logo.svg to /favicon.svg.

/logo.svg (the old rounded yellow icon) was referenced only by the favicon field, so nothing else is affected. The navbar and homepage wordmarks (logo/light.svg, logo/dark.svg) are separate and unchanged. Mintlify generates the favicon size/format variants from this single SVG at build.

Part of the org-wide favicon standardization (FE-705); design direction (square, not rounded) confirmed by Bert. Companion PRs update the main website, workflows hub, and registry.

Add a dedicated favicon.svg (full-bleed ink square with the yellow C)
and point the docs.json favicon field at it instead of /logo.svg. A
square source lets each platform apply its own corner mask. logo.svg
is left in place; nothing else references it. Part of the org-wide
favicon standardization (FE-705).
@coderabbitai

coderabbitai Bot commented Jul 6, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: dbdcbb2a-a112-4ae5-b5ca-707f295e88f5

📥 Commits

Reviewing files that changed from the base of the PR and between ab6b0bc and 831d4c7.

⛔ Files ignored due to path filters (1)
  • favicon.svg is excluded by !**/*.svg
📒 Files selected for processing (1)
  • docs.json

📝 Walkthrough

Walkthrough

The favicon property in docs.json was changed from "/logo.svg" to "/favicon.svg", updating the reference used for the documentation site's favicon.

Changes

Favicon Configuration Update

Layer / File(s) Summary
Update favicon reference
docs.json
The favicon field is changed from "/logo.svg" to "/favicon.svg".

Compact Metadata

  • Files changed: 1
  • Lines changed: +1/-1
  • Estimated review effort: Medium

No sequence diagram needed here — this ico-nic change is too svelte for a flowchart, just a tiny SVG-swap, no need to gawk! 🧚‍♂️

🚥 Pre-merge checks | ✅ 2
✅ Passed checks (2 passed)
Check name Status Explanation
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/square-favicon
✨ Simplify code
  • Create PR with simplified code
  • Commit simplified code in branch fix/square-favicon

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@comfyui-wiki
comfyui-wiki merged commit 71abdc4 into main Jul 7, 2026
7 checks passed
@github-actions
github-actions Bot deleted the fix/square-favicon branch July 7, 2026 04:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants