Skip to content

docs(README): use absolute GitHub URLs for hero logo#177

Merged
JackChen-me merged 2 commits into
mainfrom
docs/readme-hero-logo-absolute-url
Apr 25, 2026
Merged

docs(README): use absolute GitHub URLs for hero logo#177
JackChen-me merged 2 commits into
mainfrom
docs/readme-hero-logo-absolute-url

Conversation

@JackChen-me
Copy link
Copy Markdown
Member

Summary

Replace the relative .github/brand/logo-mark-*.svg paths in the hero <picture> (English + 中文) with absolute raw.githubusercontent.com/.../main/... URLs.

Why

package.json files ships only ["dist", "docs", "README.md", "LICENSE"].github/ is not in the npm tarball, so the npm-published README renders the hero logo as broken images. Switching to absolute URLs on main fixes npm without bloating the published package.

Test plan

  • curl -I https://raw.githubusercontent.com/JackChen-me/open-multi-agent/main/.github/brand/logo-mark-light.svg → 200 (assets landed on main via docs(README): refresh positioning, branding, and section layout #176)
  • Diff is exactly 6 lines per README, only the hero <source> and <img> srcset/src fields
  • GitHub web view + local IDE preview unchanged (raw URLs resolve identically there too)
  • Next npm release picks this up; npmjs.com hero renders correctly

The .github/ directory is not in package.json `files`, so npm-published
READMEs render the hero <picture> as broken images. Switch the three
sources (English + Chinese) to raw.githubusercontent.com on main so
the npm documentation matches the GitHub experience without bloating
the shipped tarball with brand assets.

The previous PR (#176) landed the brand assets on main, so these
absolute URLs now resolve. Verified raw URL returns 200.
Replace the `---` horizontal rule between the language switcher and
the intro paragraph with a `<br />` — same visual gap, no heavy line
on top of the `<h1>` underline GitHub auto-renders. Add `<br />`
before the logo and between the logo and `<h1>` so the hero block
isn't crammed against the top edge or the title.
@JackChen-me JackChen-me merged commit e549130 into main Apr 25, 2026
4 checks passed
@JackChen-me JackChen-me deleted the docs/readme-hero-logo-absolute-url branch April 25, 2026 19:34
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.

1 participant