Skip to content

Improve theme consistency, mobile nav, and MatrixRain color theming#54

Draft
Copilot wants to merge 2 commits intomainfrom
copilot/improve-codebase-and-site
Draft

Improve theme consistency, mobile nav, and MatrixRain color theming#54
Copilot wants to merge 2 commits intomainfrom
copilot/improve-codebase-and-site

Conversation

Copy link
Contributor

Copilot AI commented Mar 13, 2026

HireMe.tsx bypassed the design system with hardcoded Tailwind green classes, breaking theme toggling site-wide. Now.tsx read the persisted theme but never applied it to the DOM. The main nav overflowed on mobile.

Changes

  • HireMe.tsx — Replace all hardcoded text-green-400 / bg-black / border-green-500 with CSS-variable tokens (text-primary, bg-background, terminal-border, etc.). Add theme toggle (Sun/Moon) with the same useEffect pattern as Index.tsx. Fix inaccurate footer: "Built with Python. Deployed with Vercel""Built with React + Tailwind. Deployed on Netlify."

  • Now.tsx — Apply/remove the light class on documentElement on mount so the stored theme is actually reflected in the DOM (was only used to gate <MatrixRain /> rendering).

  • Index.tsx — Add mobile-responsive nav: desktop links use hidden md:flex; a Menu/X hamburger toggle (md:hidden) expands an inline dropdown with all nav items, closing on link click.

  • MatrixRain.tsx — Read --primary from getComputedStyle at draw time instead of hardcoding #00ff00, so rain color follows the active theme. Extract fallback to FALLBACK_PRIMARY_COLOR constant.

// Before
ctx.fillStyle = '#00ff00';

// After
const getPrimaryColor = () => {
  const raw = getComputedStyle(document.documentElement)
    .getPropertyValue('--primary').trim();
  return raw ? `hsl(${raw})` : FALLBACK_PRIMARY_COLOR;
};
ctx.fillStyle = getPrimaryColor();

💬 Send tasks to Copilot coding agent from Slack and Teams to turn conversations into code. Copilot posts an update in your thread when it's finished.

@vercel
Copy link

vercel bot commented Mar 13, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
ian-web-forge Ready Ready Preview, Comment Mar 13, 2026 8:26pm

@netlify
Copy link

netlify bot commented Mar 13, 2026

Deploy Preview for ianalloway ready!

Name Link
🔨 Latest commit fe60ec5
🔍 Latest deploy log https://app.netlify.com/projects/ianalloway/deploys/69b472d1ce662f0008c0e6b6
😎 Deploy Preview https://deploy-preview-54--ianalloway.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.

…ariable color, fix HireMe footer

- Add mobile hamburger nav to Index.tsx (hidden md:flex desktop, md:hidden hamburger)
- Fix Now.tsx to apply light class to documentElement on mount
- Refactor HireMe.tsx: replace hardcoded green Tailwind colors with CSS-variable design tokens, add theme toggle, fix footer (React+Tailwind, Netlify)
- MatrixRain: read --primary CSS variable at draw time; extract FALLBACK_PRIMARY_COLOR constant

Co-authored-by: ianalloway <200005377+ianalloway@users.noreply.github.com>
Copilot AI changed the title [WIP] Improve codebase and website performance Improve theme consistency, mobile nav, and MatrixRain color theming Mar 13, 2026
Copilot AI requested a review from ianalloway March 13, 2026 20:26
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