-
Notifications
You must be signed in to change notification settings - Fork 135
Add changes for new branding #2478
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
✅ Deploy Preview for calico-docs-preview-next ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
✅ Deploy Preview succeeded!
To edit notification comments on pull requests, go to your Netlify project configuration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR updates the branding assets for the Calico documentation website, introducing new 2026 logos for both Tigera and Calico brands. The changes aim to refresh the visual identity across the documentation site.
Changes:
- Added new brand assets: PNG and SVG logo files for Tigera and Calico with "2026" branding
- Updated configuration to reference the new logo files in docusaurus.config.js
- Removed image properties and rendering from product comparison and product card components
- Updated data files to reference new badge images
- Added test skipList entry for the new badge image
Reviewed changes
Copilot reviewed 7 out of 13 changed files in this pull request and generated 12 comments.
Show a summary per file
| File | Description |
|---|---|
| static/img/tigera-logo-2026.png | New Tigera logo PNG for 2026 branding |
| static/img/Tigera-logo-2026-white-text.svg | New Tigera logo SVG with white text |
| static/img/Tigera-logo-2026-black-text.svg | New Tigera logo SVG with black text |
| static/img/Calico-logo-2026-black-text.svg | New Calico logo SVG with black text |
| static/img/Calico-logo-2026-badge.svg | New Calico badge logo SVG |
| static/img/Calico-logo-2026-badge.png | New Calico badge logo PNG |
| src/new/data/selectDocsInfo.ts | Updated image references to use new Calico badge logo |
| src/new/data/compareInfo.ts | Removed image property from product comparison data |
| src/new/components/ProductComparison/index.tsx | Removed image rendering code from comparison table |
| src/new/components/CalicoProducts/index.tsx | Removed image-related properties from CalicoProduct type |
| src/new/components/CalicoProductCard/index.tsx | Removed image rendering section from product cards |
| docusaurus.config.js | Updated favicon and logo references to use new 2026 branding assets |
| tests/crawler.test.js | Added new badge image URL to crawler test skipList |
|
|
||
| export type CalicoProduct = { | ||
| title: string, | ||
| description: string, |
Copilot
AI
Jan 23, 2026
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The CalicoProduct type definition has been updated to remove imageSrc, alt, and imageSize properties. However, the CalicoProductCard component that consumes this type still attempts to destructure these properties in its props. This will cause TypeScript compilation errors. The component file should be updated to remove references to these removed properties.
| description: string, | |
| description: string, | |
| imageSrc?: string, | |
| alt?: string, | |
| imageSize?: string, |
| title: 'Calico Open Source', | ||
| costSupport: ['Free, self-managed'], | ||
| target: 'Users', | ||
| image: 'img/calico-logo-badge.svg', | ||
| alt: 'Calico Open Source logo', |
Copilot
AI
Jan 23, 2026
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The image property has been removed from the data but the TypeScript type definition likely still includes this property. If there's a type definition for the bestFit array items, it should be updated to reflect that the image property is no longer required. This ensures type safety and prevents confusion about which properties are expected.
| { | ||
| title: 'Calico Enterprise', | ||
| costSupport: ['Paid, self-managed'], | ||
| image: '/img/calico-enterprise-logo-badge.svg', | ||
| alt: 'Calico Enterprise logo', |
Copilot
AI
Jan 23, 2026
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The image property has been removed from the Calico Enterprise data object. Similar to the Calico Open Source entry, if there's a type definition for these data objects, it should be updated to reflect that the image property is no longer required.
| { | ||
| title: 'Calico Cloud', | ||
| costSupport: ['Pay-as-you-go, SaaS'], | ||
| image: '/img/calico-cloud-logo-badge.svg', | ||
| alt: 'Calico Cloud logo', |
Copilot
AI
Jan 23, 2026
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The image property has been removed from the Calico Cloud data object. Consistent with the other entries, if there's a type definition for these data objects, it should be updated to reflect that the image property is no longer required.
| onBrokenAnchors: 'ignore', | ||
| onBrokenLinks: 'throw', | ||
| favicon: 'img/favicon.png', | ||
| favicon: 'img/calico-logo-2026-badge.png', |
Copilot
AI
Jan 23, 2026
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The file reference uses lowercase calico-logo-2026-badge.png but the actual filename added in this PR is Calico-logo-2026-badge.png (with capital C). This case mismatch will cause the favicon to fail loading on case-sensitive file systems (like Linux servers). The reference should match the exact case of the filename, or the filename should be renamed to use lowercase to match existing naming conventions in the repository.
| favicon: 'img/calico-logo-2026-badge.png', | |
| favicon: 'img/Calico-logo-2026-badge.png', |
| logo: { | ||
| src: 'img/tigera-logo-black.png', | ||
| srcDark: 'img/tigera-logo-white.png', | ||
| src: 'img/tigera-logo-2026-black-text.svg', |
Copilot
AI
Jan 23, 2026
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The file reference uses lowercase tigera-logo-2026-black-text.svg but the actual filename added in this PR is Tigera-logo-2026-black-text.svg (with capital T). This case mismatch will cause the logo to fail loading on case-sensitive file systems. The reference should match the exact case of the filename.
| src: 'img/tigera-logo-2026-black-text.svg', | |
| src: 'img/Tigera-logo-2026-black-text.svg', |
| src: 'img/tigera-logo-black.png', | ||
| srcDark: 'img/tigera-logo-white.png', | ||
| src: 'img/tigera-logo-2026-black-text.svg', | ||
| srcDark: 'img/tigera-logo-2026-white-text.svg', |
Copilot
AI
Jan 23, 2026
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The file reference uses lowercase tigera-logo-2026-white-text.svg but the actual filename added in this PR is Tigera-logo-2026-white-text.svg (with capital T). This case mismatch will cause the dark mode logo to fail loading on case-sensitive file systems. The reference should match the exact case of the filename.
| srcDark: 'img/tigera-logo-2026-white-text.svg', | |
| srcDark: 'img/Tigera-logo-2026-white-text.svg', |
| title: 'Calico Open Source', | ||
| link: '/calico/latest/about/', | ||
| image: 'img/calico-logo-badge.svg', | ||
| image: 'img/Calico-logo-2026-badge.svg', |
Copilot
AI
Jan 23, 2026
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The file reference uses Calico-logo-2026-badge.svg (with capital C) which matches the actual filename added in this PR. However, this is inconsistent with the repository's existing naming convention where image files use lowercase names (e.g., calico-logo-badge.svg). Consider renaming the new image files to follow the existing lowercase convention for consistency across the codebase.
| title: 'Calico Enterprise', | ||
| link: '/calico-enterprise/latest/about/', | ||
| image: '/img/calico-enterprise-logo-badge.svg', | ||
| image: '/img/Calico-logo-2026-badge.svg', |
Copilot
AI
Jan 23, 2026
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The file reference uses Calico-logo-2026-badge.svg with a leading slash, while line 8 doesn't have a leading slash. This inconsistency could lead to path resolution issues depending on the context. Ensure consistent path formatting across all image references - either always include the leading slash or never include it.
| 'https://sysctl-explorer.net/net/ipv4/fib_multipath_hash_policy/', | ||
| 'https://calico-docs.mcp.kapa.ai' | ||
| 'https://calico-docs.mcp.kapa.ai', | ||
| 'https://docs.tigera.io/img/calico-logo-2026-badge.png' |
Copilot
AI
Jan 23, 2026
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This skipList entry uses lowercase calico-logo-2026-badge.png but the actual filename added in this PR is Calico-logo-2026-badge.png (with capital C). If this URL is meant to skip link checking for the new favicon, the case should match the actual filename. Otherwise, the link checker may not properly skip this URL.
| 'https://docs.tigera.io/img/calico-logo-2026-badge.png' | |
| 'https://docs.tigera.io/img/Calico-logo-2026-badge.png' |

Product Version(s):
Issue:
Link to docs preview:
SME review:
DOCS review:
Additional information:
Merge checklist: