Centralize branding assets#18806
Merged
Merged
Conversation
Contributor
Deploy preview
|
Contributor
|
Vale prose linter → found 48 errors, 73 warnings, 20 suggestions in your markdown Full report → Copy the linter results into an LLM to batch-fix issues. Linter being weird? Update the rules!
|
| Line | Severity | Message | Rule |
|---|---|---|---|
| 7:53 | warning | Use 'PostHog' instead of 'posthog'. | Vale.Terms |
| 7:86 | warning | Use 'PostHog' instead of 'posthog'. | Vale.Terms |
| 15:189 | error | Hi, Andy here... use an en dash ( – ) with spaces. On Mac, holding down the Option and hyphen key will give you an en dash. | PostHogBase.EnDash |
| 17:72 | error | Hi, Andy here... use an en dash ( – ) with spaces. On Mac, holding down the Option and hyphen key will give you an en dash. | PostHogBase.EnDash |
| 19:65 | error | Hi, Andy here... use an en dash ( – ) with spaces. On Mac, holding down the Option and hyphen key will give you an en dash. | PostHogBase.EnDash |
| 23:172 | error | Hi, Andy here... use an en dash ( – ) with spaces. On Mac, holding down the Option and hyphen key will give you an en dash. | PostHogBase.EnDash |
| 31:230 | error | Hi, Andy here... use an en dash ( – ) with spaces. On Mac, holding down the Option and hyphen key will give you an en dash. | PostHogBase.EnDash |
| 41:69 | error | Hi, Andy here... use an en dash ( – ) with spaces. On Mac, holding down the Option and hyphen key will give you an en dash. | PostHogBase.EnDash |
| 41:82 | warning | 'Anli' is a possible misspelling. | PostHogBase.Spelling |
contents/handbook/brand/assets.md — 9 errors, 15 warnings, 11 suggestions
| Line | Severity | Message | Rule |
|---|---|---|---|
| 30:44 | warning | Use 'PostHog' instead of 'posthog'. | Vale.Terms |
| 33:19 | warning | 'logomark' is a possible misspelling. | PostHogBase.Spelling |
| 33:47 | warning | 'wordmark' is a possible misspelling. | PostHogBase.Spelling |
| 39:104 | error | Hi, Andy here... use an en dash ( – ) with spaces. On Mac, holding down the Option and hyphen key will give you an en dash. | PostHogBase.EnDash |
| 39:146 | error | Hi, Andy here... use an en dash ( – ) with spaces. On Mac, holding down the Option and hyphen key will give you an en dash. | PostHogBase.EnDash |
| 45:27 | error | Hi, Andy here... use an en dash ( – ) with spaces. On Mac, holding down the Option and hyphen key will give you an en dash. | PostHogBase.EnDash |
| 46:76 | error | Hi, Andy here... use an en dash ( – ) with spaces. On Mac, holding down the Option and hyphen key will give you an en dash. | PostHogBase.EnDash |
| 49:15 | warning | 'logomark' is a possible misspelling. | PostHogBase.Spelling |
| 49:57 | error | Hi, Andy here... use an en dash ( – ) with spaces. On Mac, holding down the Option and hyphen key will give you an en dash. | PostHogBase.EnDash |
| 49:59 | warning | 'favicons' is a possible misspelling. | PostHogBase.Spelling |
| 50:20 | warning | 'logomark' is a possible misspelling. | PostHogBase.Spelling |
| 50:129 | warning | 'logomark's' is a possible misspelling. | PostHogBase.Spelling |
| 50:159 | error | Hi, Andy here... use an en dash ( – ) with spaces. On Mac, holding down the Option and hyphen key will give you an en dash. | PostHogBase.EnDash |
| 50:188 | warning | 'logomark' is a possible misspelling. | PostHogBase.Spelling |
| 53:34 | warning | 'logomark' is a possible misspelling. | PostHogBase.Spelling |
| 53:118 | error | Hi, Andy here... use an en dash ( – ) with spaces. On Mac, holding down the Option and hyphen key will give you an en dash. | PostHogBase.EnDash |
| 55:12 | warning | 'PNGs' is a possible misspelling. | PostHogBase.Spelling |
| 55:213 | error | Hi, Andy here... use an en dash ( – ) with spaces. On Mac, holding down the Option and hyphen key will give you an en dash. | PostHogBase.EnDash |
| 57:258 | warning | 'logomark' is a possible misspelling. | PostHogBase.Spelling |
| 100:155 | error | Hi, Andy here... use an en dash ( – ) with spaces. On Mac, holding down the Option and hyphen key will give you an en dash. | PostHogBase.EnDash |
| 129:45 | warning | 'semibold' is a possible misspelling. | PostHogBase.Spelling |
| 130:46 | warning | 'semibold' is a possible misspelling. | PostHogBase.Spelling |
| 138:72 | suggestion | Prefer 'proper nouns or they' over 'he'. | PostHogBase.Inclusivity |
| 138:86 | suggestion | Prefer 'proper nouns or their/theirs' over 'his'. | PostHogBase.Inclusivity |
| 138:90 | warning | 'hoggy' is a possible misspelling. | PostHogBase.Spelling |
| 142:26 | suggestion | Prefer 'proper nouns or they' over 'he'. | PostHogBase.Inclusivity |
| 142:96 | suggestion | Prefer 'proper nouns or their/theirs' over 'His'. | PostHogBase.Inclusivity |
| 142:198 | suggestion | Prefer 'proper nouns or their/theirs' over 'His'. | PostHogBase.Inclusivity |
| 142:232 | suggestion | Prefer 'proper nouns or their/theirs' over 'his'. | PostHogBase.Inclusivity |
| 142:302 | suggestion | Prefer 'proper nouns or their/theirs' over 'His'. | PostHogBase.Inclusivity |
| 142:335 | suggestion | Prefer 'proper nouns or their/theirs' over 'his'. | PostHogBase.Inclusivity |
| 146:1 | suggestion | Prefer 'proper nouns or they' over 'He'. | PostHogBase.Inclusivity |
| 146:44 | warning | 'monoline' is a possible misspelling. | PostHogBase.Spelling |
| 146:80 | suggestion | Prefer 'proper nouns or they' over 'He'. | PostHogBase.Inclusivity |
| 146:191 | suggestion | Prefer 'proper nouns or they' over 'he'. | PostHogBase.Inclusivity |
contents/handbook/brand/visual-identity.md — 0 errors, 17 warnings, 4 suggestions
| Line | Severity | Message | Rule |
|---|---|---|---|
| 26:20 | warning | 'wordmark' is a possible misspelling. | PostHogBase.Spelling |
| 27:21 | warning | 'wordmark' is a possible misspelling. | PostHogBase.Spelling |
| 28:3 | warning | 'Logomark' is a possible misspelling. | PostHogBase.Spelling |
| 28:71 | warning | 'favicons' is a possible misspelling. | PostHogBase.Spelling |
| 36:22 | warning | 'logomark' is a possible misspelling. | PostHogBase.Spelling |
| 40:70 | warning | 'logomark' is a possible misspelling. | PostHogBase.Spelling |
| 75:5 | warning | 'RoundHog' heading should be in sentence case, and product names should be capitalized. | PostHogBase.SentenceCase |
| 77:44 | warning | Use 'PostHog' instead of 'posthog'. | Vale.Terms |
| 82:5 | warning | 'Semibold' is a possible misspelling. | PostHogBase.Spelling |
| 85:52 | warning | 'Semibold' is a possible misspelling. | PostHogBase.Spelling |
| 120:29 | suggestion | Prefer 'proper nouns or they' over 'He'. | PostHogBase.Inclusivity |
| 120:78 | suggestion | Prefer 'proper nouns or they' over 'He'. | PostHogBase.Inclusivity |
| 120:175 | suggestion | Prefer 'proper nouns or them' over 'him'. | PostHogBase.Inclusivity |
| 120:233 | suggestion | Prefer 'proper nouns or them' over 'him'. | PostHogBase.Inclusivity |
| 218:95 | warning | 'webpages' is a possible misspelling. | PostHogBase.Spelling |
| 224:238 | warning | 'easter' is a possible misspelling. | PostHogBase.Spelling |
| 224:304 | warning | Capitalize 'Session Replay' for PostHog's product. Use 'session replay' for the general industry concept. | PostHogBase.ProductNames |
| 230:206 | warning | Capitalize 'Session Replay' for PostHog's product. Use 'session replay' for the general industry concept. | PostHogBase.ProductNames |
| 251:38 | warning | Use 'PostHog' instead of 'posthog'. | Vale.Terms |
| 255:53 | warning | 'gridlines' is a possible misspelling. | PostHogBase.Spelling |
| 263:65 | warning | Use 'PostHog' instead of 'posthog'. | Vale.Terms |
contents/handbook/marketing/events.md — 2 errors, 9 warnings, 1 suggestions
| Line | Severity | Message | Rule |
|---|---|---|---|
| 9:162 | error | Hi, Andy here... use an en dash ( – ) with spaces. On Mac, holding down the Option and hyphen key will give you an en dash. | PostHogBase.EnDash |
| 20:211 | warning | Use the Oxford comma before 'and' or 'or' in a list of three or more items. | PostHogBase.OxfordComma |
| 20:219 | warning | 'logstics' is a possible misspelling. | PostHogBase.Spelling |
| 30:10 | warning | 'Taj' is a possible misspelling. | PostHogBase.Spelling |
| 30:22 | suggestion | Prefer 'proper nouns or their/theirs' over 'his'. | PostHogBase.Inclusivity |
| 41:28 | warning | 'roundtable' is a possible misspelling. | PostHogBase.Spelling |
| 53:98 | warning | 'hyperspecific' is a possible misspelling. | PostHogBase.Spelling |
| 54:98 | warning | 'scaleups' is a possible misspelling. | PostHogBase.Spelling |
| 67:53 | warning | Use the Oxford comma before 'and' or 'or' in a list of three or more items. | PostHogBase.OxfordComma |
| 76:12 | warning | 'LARPing' is a possible misspelling. | PostHogBase.Spelling |
| 105:38 | error | Hi, Andy here... use an en dash ( – ) with spaces. On Mac, holding down the Option and hyphen key will give you an en dash. | PostHogBase.EnDash |
| 111:16 | warning | 'yappers' is a possible misspelling. | PostHogBase.Spelling |
contents/handbook/marketing/ownership.md — 2 errors, 10 warnings, 0 suggestions
| Line | Severity | Message | Rule |
|---|---|---|---|
| 17:43 | warning | 'Blitzscale' is a possible misspelling. | PostHogBase.Spelling |
| 20:25 | warning | 'Annika' is a possible misspelling. | PostHogBase.Spelling |
| 21:25 | warning | 'Annika' is a possible misspelling. | PostHogBase.Spelling |
| 22:25 | warning | 'Annika' is a possible misspelling. | PostHogBase.Spelling |
| 27:34 | warning | 'Danilo' is a possible misspelling. | PostHogBase.Spelling |
| 31:33 | warning | 'PMMs' is a possible misspelling. | PostHogBase.Spelling |
| 34:14 | warning | 'comms' is a possible misspelling. | PostHogBase.Spelling |
| 67:64 | warning | 'backlink' is a possible misspelling. | PostHogBase.Spelling |
| 81:13 | warning | 'influencer' is a possible misspelling. | PostHogBase.Spelling |
| 113:27 | error | Hi, Andy here... use an en dash ( – ) with spaces. On Mac, holding down the Option and hyphen key will give you an en dash. | PostHogBase.EnDash |
| 113:77 | error | Hi, Andy here... use an en dash ( – ) with spaces. On Mac, holding down the Option and hyphen key will give you an en dash. | PostHogBase.EnDash |
| 137:22 | warning | Use 'PostHog' instead of 'posthog'. | Vale.Terms |
contents/handbook/marketing/speaker-guide.md — 21 errors, 5 warnings, 2 suggestions
| Line | Severity | Message | Rule |
|---|---|---|---|
| 7:216 | error | Hi, Andy here... use an en dash ( – ) with spaces. On Mac, holding down the Option and hyphen key will give you an en dash. | PostHogBase.EnDash |
| 15:78 | warning | Use 'ClickHouse' instead of 'Clickhouse'. | Vale.Terms |
| 15:322 | error | Hi, Andy here... use an en dash ( – ) with spaces. On Mac, holding down the Option and hyphen key will give you an en dash. | PostHogBase.EnDash |
| 42:9 | warning | 'Hightower' is a possible misspelling. | PostHogBase.Spelling |
| 42:66 | error | Hi, Andy here... use an en dash ( – ) with spaces. On Mac, holding down the Option and hyphen key will give you an en dash. | PostHogBase.EnDash |
| 42:136 | suggestion | Prefer 'proper nouns or they' over 'He'. | PostHogBase.Inclusivity |
| 42:146 | suggestion | Prefer 'proper nouns or their/theirs' over 'his'. | PostHogBase.Inclusivity |
| 63:24 | error | Hi, Andy here... use an en dash ( – ) with spaces. On Mac, holding down the Option and hyphen key will give you an en dash. | PostHogBase.EnDash |
| 68:39 | error | Hi, Andy here... use an en dash ( – ) with spaces. On Mac, holding down the Option and hyphen key will give you an en dash. | PostHogBase.EnDash |
| 71:32 | error | Hi, Andy here... use an en dash ( – ) with spaces. On Mac, holding down the Option and hyphen key will give you an en dash. | PostHogBase.EnDash |
| 73:32 | error | Hi, Andy here... use an en dash ( – ) with spaces. On Mac, holding down the Option and hyphen key will give you an en dash. | PostHogBase.EnDash |
| 89:101 | warning | 'Hoggies' is a possible misspelling. | PostHogBase.Spelling |
| 93:93 | warning | 'Brasseur' is a possible misspelling. | PostHogBase.Spelling |
| 97:146 | error | Hi, Andy here... use an en dash ( – ) with spaces. On Mac, holding down the Option and hyphen key will give you an en dash. | PostHogBase.EnDash |
| 105:102 | warning | 'Sumbit' is a possible misspelling. | PostHogBase.Spelling |
| 111:188 | error | Hi, Andy here... use an en dash ( – ) with spaces. On Mac, holding down the Option and hyphen key will give you an en dash. | PostHogBase.EnDash |
| 117:1 | error | Hi, Andy here... use an en dash ( – ) with spaces. On Mac, holding down the Option and hyphen key will give you an en dash. | PostHogBase.EnDash |
| 118:52 | error | Hi, Andy here... use an en dash ( – ) with spaces. On Mac, holding down the Option and hyphen key will give you an en dash. | PostHogBase.EnDash |
| 119:31 | error | Hi, Andy here... use an en dash ( – ) with spaces. On Mac, holding down the Option and hyphen key will give you an en dash. | PostHogBase.EnDash |
| 120:25 | error | Hi, Andy here... use an en dash ( – ) with spaces. On Mac, holding down the Option and hyphen key will give you an en dash. | PostHogBase.EnDash |
| 126:11 | error | Hi, Andy here... use an en dash ( – ) with spaces. On Mac, holding down the Option and hyphen key will give you an en dash. | PostHogBase.EnDash |
| 127:10 | error | Hi, Andy here... use an en dash ( – ) with spaces. On Mac, holding down the Option and hyphen key will give you an en dash. | PostHogBase.EnDash |
| 128:12 | error | Hi, Andy here... use an en dash ( – ) with spaces. On Mac, holding down the Option and hyphen key will give you an en dash. | PostHogBase.EnDash |
| 129:12 | error | Hi, Andy here... use an en dash ( – ) with spaces. On Mac, holding down the Option and hyphen key will give you an en dash. | PostHogBase.EnDash |
| 130:10 | error | Hi, Andy here... use an en dash ( – ) with spaces. On Mac, holding down the Option and hyphen key will give you an en dash. | PostHogBase.EnDash |
| 131:10 | error | Hi, Andy here... use an en dash ( – ) with spaces. On Mac, holding down the Option and hyphen key will give you an en dash. | PostHogBase.EnDash |
| 132:10 | error | Hi, Andy here... use an en dash ( – ) with spaces. On Mac, holding down the Option and hyphen key will give you an en dash. | PostHogBase.EnDash |
| 133:10 | error | Hi, Andy here... use an en dash ( – ) with spaces. On Mac, holding down the Option and hyphen key will give you an en dash. | PostHogBase.EnDash |
contents/library/ops/registering-trademarks.md — 6 errors, 3 warnings, 0 suggestions
| Line | Severity | Message | Rule |
|---|---|---|---|
| 11:53 | warning | Use 'PostHog' instead of 'posthog'. | Vale.Terms |
| 11:94 | warning | Use 'PostHog' instead of 'posthog'. | Vale.Terms |
| 17:189 | error | Hi, Andy here... use an en dash ( – ) with spaces. On Mac, holding down the Option and hyphen key will give you an en dash. | PostHogBase.EnDash |
| 19:72 | error | Hi, Andy here... use an en dash ( – ) with spaces. On Mac, holding down the Option and hyphen key will give you an en dash. | PostHogBase.EnDash |
| 21:65 | error | Hi, Andy here... use an en dash ( – ) with spaces. On Mac, holding down the Option and hyphen key will give you an en dash. | PostHogBase.EnDash |
| 25:172 | error | Hi, Andy here... use an en dash ( – ) with spaces. On Mac, holding down the Option and hyphen key will give you an en dash. | PostHogBase.EnDash |
| 33:230 | error | Hi, Andy here... use an en dash ( – ) with spaces. On Mac, holding down the Option and hyphen key will give you an en dash. | PostHogBase.EnDash |
| 40:69 | error | Hi, Andy here... use an en dash ( – ) with spaces. On Mac, holding down the Option and hyphen key will give you an en dash. | PostHogBase.EnDash |
| 40:82 | warning | 'Anli' is a possible misspelling. | PostHogBase.Spelling |
contents/media-contents.mdx — 0 errors, 8 warnings, 2 suggestions
| Line | Severity | Message | Rule |
|---|---|---|---|
| 10:50 | warning | Use 'PostHog' instead of 'posthog'. | Vale.Terms |
| 20:84 | warning | 'Combinator's' is a possible misspelling. | PostHogBase.Spelling |
| 20:344 | warning | 'Hykes' is a possible misspelling. | PostHogBase.Spelling |
| 33:112 | warning | 'Combinator' is a possible misspelling. | PostHogBase.Spelling |
| 34:164 | warning | 'Formus' is a possible misspelling. | PostHogBase.Spelling |
| 35:189 | warning | 'Formus' is a possible misspelling. | PostHogBase.Spelling |
| 45:12 | warning | 'headshots' is a possible misspelling. | PostHogBase.Spelling |
| 71:43 | suggestion | Prefer 'proper nouns or they' over 'he'. | PostHogBase.Inclusivity |
| 71:60 | suggestion | Prefer 'proper nouns or their/theirs' over 'his'. | PostHogBase.Inclusivity |
| 71:64 | warning | Use American English. Use 'favorite' instead of 'favourite'. | PostHogBase.AmericanEnglish |
contents/tutorials/how-to-connect-posthog-and-notion-with-zapier.md — 1 errors, 1 warnings, 0 suggestions
| Line | Severity | Message | Rule |
|---|---|---|---|
| 49:57 | warning | Capitalize 'Workflows' for PostHog's product. Use 'workflows' for the general industry concept. | PostHogBase.ProductNames |
| 62:81 | error | Hi, Andy here... use an en dash ( – ) with spaces. On Mac, holding down the Option and hyphen key will give you an en dash. | PostHogBase.EnDash |
src/components/BrandLogos/README.md — 1 errors, 2 warnings, 0 suggestions
| Line | Severity | Message | Rule |
|---|---|---|---|
| 4:76 | error | Hi, Andy here... use an en dash ( – ) with spaces. On Mac, holding down the Option and hyphen key will give you an en dash. | PostHogBase.EnDash |
| 14:22 | warning | 'shortcode' is a possible misspelling. | PostHogBase.Spelling |
| 24:33 | warning | 'logomarks' is a possible misspelling. | PostHogBase.Spelling |
Contributor
Bundle reportTotal JS (gzip)6.74 MiB (-12.1 KiB / -0.2%) Largest changed named chunks
Eager graph (modules shipped in each entrypoint's initial chunks)
Largest modules in the
|
| Module | Size |
|---|---|
./src/data/mcp-tools.json |
897.2 KiB |
css ./node_modules/.pnpm/css-loader@5.2.7_webpack@5.101.3/node_modules/css-loader/dist/cjs.js??ruleSet[1].rules[8].oneOf[1].use[1]!./node_modules/.pnpm/postcss-loader@4.3.0_postcss@8.5.6_webpack@5.101.3/node_modules/postcss-loader/dist/cjs.js??ruleSet[1].rules[8].oneOf[1].use[2]!./src/styles/global.css |
747.3 KiB |
./src/components/Stickers/Stickers.tsx |
696.4 KiB |
./node_modules/.pnpm/@radix-ui+react-icons@1.3.2_react@18.3.1/node_modules/@radix-ui/react-icons/dist/react-icons.esm.js |
481.4 KiB |
./node_modules/.pnpm/rehype-raw@7.0.0/node_modules/rehype-raw/lib/index.js + 29 modules |
395.1 KiB |
./node_modules/.pnpm/@posthog+icons@0.36.6_react-dom@18.3.1_react@18.3.1__react@18.3.1/node_modules/@posthog/icons/dist/posthog-icons.cjs.js |
364.8 KiB |
./src/hooks/useCustomers.tsx + 54 modules |
355.1 KiB |
./node_modules/.pnpm/@posthog+icons@0.36.6_react-dom@18.3.1_react@18.3.1__react@18.3.1/node_modules/@posthog/icons/dist/posthog-icons.es.js |
354.8 KiB |
./node_modules/.pnpm/react-markdown@8.0.7_@types+react@16.14.66_react@18.3.1/node_modules/react-markdown/lib/react-markdown.js + 88 modules |
351.4 KiB |
./node_modules/.pnpm/cloudinary-core@2.14.0_lodash@4.17.21/node_modules/cloudinary-core/cloudinary-core.js |
281.9 KiB |
./src/components/ProductComparisonTable/index.tsx + 120 modules |
278.6 KiB |
./src/components/SearchUI/index.tsx + 87 modules |
272.1 KiB |
./node_modules/.pnpm/framer-motion@10.18.0_react-dom@18.3.1_react@18.3.1__react@18.3.1/node_modules/framer-motion/dist/es/render/dom/motion.mjs + 109 modules |
253.9 KiB |
./node_modules/.pnpm/d3@7.9.0/node_modules/d3/src/index.js + 208 modules |
247.4 KiB |
./src/components/Pricing/PricingSlider/Slider.tsx + 87 modules |
239.9 KiB |
Eager-graph budgets are report-only until a baseline is established. Sizes are gzip of public/**/*.js; eager size is webpack module source bytes for the modules actually shipped in the entrypoint's initial chunks (post-tree-shake).
- Updated package.json to include new prebuild scripts for generating brand assets. - Changed references in documentation and components from the old brand assets path to the new path. - Removed the obsolete company brand assets page and updated links across the handbook. - Deleted the sync-brand-assets script as it is no longer needed with the new asset management approach.
- Removed references to the `Logo` component from `@posthog/brand/logo` in multiple files. - Updated logo usage to directly reference SVG paths instead of the React component. - Adjusted related logic in `useCustomers` and other components to accommodate the changes.
Contributor
🦔 ReviewHog reviewed this pull requestFound 0 must fix, 2 should fix, 1 consider. Published 3 findings (view the review). |
…centralize-branding
Contributor
|
ReviewHog Alpha 🦔 If you find any issues helpful - please reply "valid", "invalid", etc., for evaluation purposes 🙏 |
Contributor
There was a problem hiding this comment.
ReviewHog Report
Infrastructure
Issues: 1 issue
Files (21)
scripts/generate-brand-assets.mjspackage.jsonsrc/components/BrandLogos/index.tsxsrc/components/BrandLogos/README.mdsrc/components/Logo/index.tsxsrc/components/Home/images/Logomark.jssrc/components/CustomerLogos/PostHogLogo.tsxstatic/brand/posthog-logo-black-padded.svgstatic/brand/posthog-logo-black.svgstatic/brand/posthog-logo-padded.svgstatic/brand/posthog-logo-stacked.svgstatic/brand/posthog-logo-white-padded.svgstatic/brand/posthog-logo-white.svgstatic/brand/posthog-logo.svgstatic/brand/posthog-logomark-padded.svgstatic/brand/posthog-logomark-square.svgstatic/brand/posthog-logomark.svgsrc/data/typecaast/cursor-broken-link.jsonsrc/data/typecaast/slack-ask-posthog.jsonsrc/data/typecaast/slack-broken-link.jsonsrc/data/typecaast/slack-signals-loading.json
What were the main changes
- New scripts/generate-brand-assets.mjs materializes stable /brand/*.svg and PNG variants from the canonical @posthog/brand package before build/start (prebuild lifecycle hooks added in package.json), including a new square logomark canvas for avatar/integration use cases
- Removed the hand-maintained src/components/Logo, Home/images/Logomark, and CustomerLogos/PostHogLogo implementations now that @posthog/brand/logo is the single source of truth
- BrandLogos/index.tsx and its README updated to support rendering/downloading square logomark variants (standard + light) alongside existing lockups
- Regenerated static/brand SVG outputs (including new posthog-logomark-square.svg) to match the new generation pipeline
- Chatbot mockup fixtures (typecaast JSON) now reference the new square logomark asset for avatars
Frontend
Issues: 2 issues
Files (44)
src/components/About/v2/Letterhead.tsxsrc/components/Breadcrumbs/index.tsxsrc/components/Careers/FounderNote/index.tsxsrc/components/Careers/Transparency/index.tsxsrc/components/ColoringBook/index.jssrc/components/EventGraphic/index.tsxsrc/components/Footer/Footer.tsxsrc/components/Home/Control/index.tsxsrc/components/Home/HeroCarousel/slides.tsxsrc/components/Home/Index.tsxsrc/components/Home/Sections/Hero.tsxsrc/components/Korean/KoreanTaskBarMenu/menuData.tsxsrc/components/Layout/Fonts.csssrc/components/MainNav/index.tsxsrc/components/Merch/OrderHistory.tsxsrc/components/Presentation/Templates/PricingTemplate.tsxsrc/components/Presentation/Templates/StackedTemplate.tsxsrc/components/Presentation/Utilities/Logos.tsxsrc/components/Pricing/CloudVsSelfHost/index.tsxsrc/components/Product/Comparison.tsxsrc/components/ProductComparisonTable/index.tsxsrc/components/Products/Competitor/VsPostHog.tsxsrc/components/Products/ReaderViewProduct/templates/ComparisonSummary.tsxsrc/components/Products/Slides/ComparisonSummarySlide.tsxsrc/components/Products/Slides/CustomersSlide.tsxsrc/components/Products/Slides/TracesComparisonTable.tsxsrc/components/SignUp/Intro.jssrc/components/SignUp/Layout.jssrc/components/Squeak/components/Question.tsxsrc/components/Squeak/components/Reply.tsxsrc/components/Start/index.jssrc/components/Startups/StartupProgram.tsxsrc/components/TaskBarMenu/menuData.tsxsrc/components/Whitepaper/index.jssrc/hooks/useCustomers.tsxsrc/pages/about.tsxsrc/pages/components/index.tsxsrc/pages/cool-tech-jobs.tsxsrc/pages/dpa.tsxsrc/pages/ko/_KoreanHome.tsxsrc/pages/posthug.tsxsrc/pages/r/ai-observability.tsxsrc/pages/sales.tsxsrc/pages/vibe-check/index.tsx
What were the main changes
- Site-wide migration of ~40 components/pages from the local
components/Logo(props: wordmark, fill, stacked, code) to@posthog/brand/logo(props: layout, variant, color, width) - Fonts.css font-face src URLs updated to load RoundHog fonts from the @posthog/brand package instead of Cloudinary, with adjusted font-weight scale
- Removed unused/dead
Logoimports left over from earlier refactors in several files (FounderNote, ColoringBook, Home/Index, Presentation templates, Start, Whitepaper, about.tsx) - useCustomers.tsx replaces the custom PostHogLogo SVG component with static /brand logo URLs and introduces a shared CustomerLogo type
- cool-tech-jobs.tsx, dpa.tsx and r/ai-observability.tsx swap bespoke PostHog logo components/markup for direct /brand static asset references with proper alt text
- pages/components/index.tsx demo/reference page updated to reflect the new Logo API (layout instead of wordmark/stacked/code)
- TracesComparisonTable.tsx also refactors the featureRow helper signature alongside the Logo import update
ianmatson
marked this pull request as ready for review
July 22, 2026 21:46
5 tasks
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.
Changes
Does quite a few things:
@posthog/brand/logo, replaces all instances with updated brandRoundHogfrom the@posthog/brandpackage directly@posthog/brandpackage rather that using static asset links, but static asset links are still used by external and internal sources and should be maintained. This keeps them in sync with the brand package.Add screenshots or screen recordings for visual / UI-focused changes.
Checklist
The routes with additional page-specific logo changes are given below, I reviewed each one manually to ensure they render properly.
Routes whose social/OG metadata uses a regenerated logo image: