Skip to content

Conversation

@ctauchen
Copy link
Collaborator

Product Version(s):

Issue:

Link to docs preview:

SME review:

  • An SME has approved this change.

DOCS review:

  • A member of the docs team has approved this change.

Additional information:

Merge checklist:

  • Deploy preview inspected wherever changes were made
  • Build completed successfully
  • Test have passed

@ctauchen ctauchen requested a review from a team as a code owner January 23, 2026 11:53
Copilot AI review requested due to automatic review settings January 23, 2026 11:53
@netlify
Copy link

netlify bot commented Jan 23, 2026

Deploy Preview for calico-docs-preview-next ready!

Name Link
🔨 Latest commit 7a10a3e
🔍 Latest deploy log https://app.netlify.com/projects/calico-docs-preview-next/deploys/6973920521ddc00008a855d4
😎 Deploy Preview https://deploy-preview-2478--calico-docs-preview-next.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@netlify
Copy link

netlify bot commented Jan 23, 2026

Deploy Preview succeeded!

Name Link
🔨 Latest commit 7a10a3e
🔍 Latest deploy log https://app.netlify.com/projects/tigera/deploys/6973920521ddc00008a855cf
😎 Deploy Preview https://deploy-preview-2478--tigera.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 74 (🟢 up 5 from production)
Accessibility: 98 (no change from production)
Best Practices: 92 (no change from production)
SEO: 100 (no change from production)
PWA: -
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link
Contributor

Copilot AI left a 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,
Copy link

Copilot AI Jan 23, 2026

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.

Suggested change
description: string,
description: string,
imageSrc?: string,
alt?: string,
imageSize?: string,

Copilot uses AI. Check for mistakes.
Comment on lines 44 to 47
title: 'Calico Open Source',
costSupport: ['Free, self-managed'],
target: 'Users',
image: 'img/calico-logo-badge.svg',
alt: 'Calico Open Source logo',
Copy link

Copilot AI Jan 23, 2026

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.

Copilot uses AI. Check for mistakes.
Comment on lines 51 to 54
{
title: 'Calico Enterprise',
costSupport: ['Paid, self-managed'],
image: '/img/calico-enterprise-logo-badge.svg',
alt: 'Calico Enterprise logo',
Copy link

Copilot AI Jan 23, 2026

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.

Copilot uses AI. Check for mistakes.
Comment on lines 65 to 68
{
title: 'Calico Cloud',
costSupport: ['Pay-as-you-go, SaaS'],
image: '/img/calico-cloud-logo-badge.svg',
alt: 'Calico Cloud logo',
Copy link

Copilot AI Jan 23, 2026

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.

Copilot uses AI. Check for mistakes.
onBrokenAnchors: 'ignore',
onBrokenLinks: 'throw',
favicon: 'img/favicon.png',
favicon: 'img/calico-logo-2026-badge.png',
Copy link

Copilot AI Jan 23, 2026

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.

Suggested change
favicon: 'img/calico-logo-2026-badge.png',
favicon: 'img/Calico-logo-2026-badge.png',

Copilot uses AI. Check for mistakes.
logo: {
src: 'img/tigera-logo-black.png',
srcDark: 'img/tigera-logo-white.png',
src: 'img/tigera-logo-2026-black-text.svg',
Copy link

Copilot AI Jan 23, 2026

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.

Suggested change
src: 'img/tigera-logo-2026-black-text.svg',
src: 'img/Tigera-logo-2026-black-text.svg',

Copilot uses AI. Check for mistakes.
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',
Copy link

Copilot AI Jan 23, 2026

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.

Suggested change
srcDark: 'img/tigera-logo-2026-white-text.svg',
srcDark: 'img/Tigera-logo-2026-white-text.svg',

Copilot uses AI. Check for mistakes.
title: 'Calico Open Source',
link: '/calico/latest/about/',
image: 'img/calico-logo-badge.svg',
image: 'img/Calico-logo-2026-badge.svg',
Copy link

Copilot AI Jan 23, 2026

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.

Copilot uses AI. Check for mistakes.
title: 'Calico Enterprise',
link: '/calico-enterprise/latest/about/',
image: '/img/calico-enterprise-logo-badge.svg',
image: '/img/Calico-logo-2026-badge.svg',
Copy link

Copilot AI Jan 23, 2026

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.

Copilot uses AI. Check for mistakes.
'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'
Copy link

Copilot AI Jan 23, 2026

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.

Suggested change
'https://docs.tigera.io/img/calico-logo-2026-badge.png'
'https://docs.tigera.io/img/Calico-logo-2026-badge.png'

Copilot uses AI. Check for mistakes.
@ctauchen ctauchen changed the title Adds changes for new branding Add changes for new branding Jan 23, 2026
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