Skip to content

Use the TFMC logo in place of the word "TFMC" in the header and landing pages - #90

Merged
ryanbarlow97 merged 2 commits into
mainfrom
header-logo
Oct 6, 2026
Merged

ryanbarlow97 merged 2 commits into
mainfrom
header-logo

Conversation

@ryanbarlow97

@ryanbarlow97 ryanbarlow97 commented Oct 6, 2026 •

Copy link
Copy Markdown
Contributor

The site's header and landing pages showed "TFMC" as text in Fraunces. They now show the round TFMC crest:

  • Desktop bar and mobile header: 40 px logo that links home.
  • Slide-out mobile menu: 96 px logo at the top.
  • Home page heading: 160–256 px logo, depending on screen width.
  • In-development landing page (NEXT_PUBLIC_SITE_DEV_GATE=1): 160–208 px logo.

The home links are labelled "TFMC home", and the headings keep "TFMC" as their alt text. The original public/logo.png is 1 MB, so the site uses smaller WebP copies: logo-256.webp (17 KB) in the header and logo-512.webp (46 KB) in the headings. Both stay sharp on high-resolution screens.

Checked with tsc -p tsconfig.build.json, the full Vitest suite (1813 passed), and headless Chromium screenshots of the header, the mobile menu, the home page and the in-development page on desktop and mobile.

🤖 Generated with Claude Code

The desktop bar and the mobile menu now link home through the round TFMC
crest. A 256 px copy of public/logo.png keeps the header light; the
original is 1 MB.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Oct 6, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: b0335781-7c2c-4b37-bab4-4160f89cb3c0
📥 Commits

Reviewing files that changed from the base of the PR and between 4a76f03 and ef4e5de.

📒 Files selected for processing (5)
  • frontend/app/components/shell/SiteHeader.tsx
  • frontend/app/components/site/DevLandingPage.tsx
  • frontend/app/page.tsx
  • frontend/public/logo-256.webp
  • frontend/public/logo-512.webp

Included review availability: This review used your included allowance. Your plan provides up to 10 included reviews per hour; 8 remain after this review.


📝 Summary

Summary by CodeRabbit

  • Style
    • Replaced the “TFMC” text branding in the site header, menu and landing pages with logo images.

Walkthrough

The site header, menu, development landing page and hub page replace TFMC text with logo images. The header and menu use /logo-256.webp; the page headings use /logo-512.webp.

Changes

Logo branding

Layer / File(s) Summary
Header logo links
frontend/app/components/shell/SiteHeader.tsx
The header and menu home links display the shared logo image at 40×40 and 96×96 pixels.
Page logo headings
frontend/app/components/site/DevLandingPage.tsx, frontend/app/page.tsx
Both page headings display TFMC logo images with alt text and responsive sizing.

Priority: ⬇️ Low

Estimated code review effort: 2 (Simple) | ~10 minutes

Change: Feature

Suggested reviewers: justinasla

Merge Risk: ⚪ Minimal · up to ef4e5

The change replaces text branding with the TFMC crest while preserving accessible labels and home navigation. No material merge-readiness risk is established.

  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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

The home page and the in-development landing page now show the crest in
place of the "TFMC" heading, with "TFMC" as its alt text. Both logo copies
are WebP: 17 KB for the header and 46 KB at 512 px for the headings.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@ryanbarlow97 ryanbarlow97 changed the title Show the TFMC logo in the site header instead of the word "TFMC" Use the TFMC logo in place of the word "TFMC" in the header and landing pages Oct 6, 2026
@ryanbarlow97
ryanbarlow97 merged commit 83fefd8 into main Oct 6, 2026
2 checks passed
@ryanbarlow97
ryanbarlow97 deleted the header-logo branch October 6, 2026 09:21
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