Skip to content

refactor(about): fold Skills into About, cut unprovable Stack entries, vary the repeated copy - #226

Merged
Aswinmcw merged 1 commit into
mainfrom
refactor/fold-skills-into-about
Sep 3, 2026
Merged

refactor(about): fold Skills into About, cut unprovable Stack entries, vary the repeated copy#226
Aswinmcw merged 1 commit into
mainfrom
refactor/fold-skills-into-about

Conversation

@Aswinmcw

@Aswinmcw Aswinmcw commented Sep 3, 2026

Copy link
Copy Markdown
Member

Summary

Option A from the UI/UX review — items 5, 6 and 7.

  • Skills section removed; its proof lines move into About. The page covered the same four themes three sections in a row (About focus areas → Skills cards → Stack). About's four focus cards now each carry the two evidence lines the Skills cards had (ttperf on PyPI, the TTNN tracker, the self-hosted services, Lighthouse scores), plus a small "see the Stack" pointer that replaces the old Skills sub-copy. The two About columns are now vertically centred against each other, since the card column grew. Page goes 8 → 7 sections; nav drops "Skills"; sections renumber 01–05.
  • Section rhythm re-laid. Removing Skills would have put Experience (ink, cyan) directly against Projects (ink, cyan). Projects is now canvas + brand accent, Stack ink, Contact canvas, Footer ink, so no two neighbours share a surface or a hue. Project cards' per-kind accents are unaffected.
  • Stack: cut Azure, VPN, Windows, Network Administration. Their descriptions were definitions ("Cloud computing services and solutions", "Virtual Private Network setup and management") with nothing on the site behind them, which contradicted the closing card's "all of these have shipped something" claim. Operating Systems (3 left) and Networking (2 left) merge into "Systems & Networking"; the grid becomes three columns of five at lg, one column below. The derived count in the closing card now reads 15.
  • Repeated copy. "AI accelerators go faster / software that runs on AI silicon" appeared in the hero, About p1, the About "Currently" line, the MulticoreWare card and the footer. Hero keeps it; About p1 now names the TT-Metal work and describes a day; the card lead and footer tagline say something different.

No new factual claims were added — every proof line points at a project or fact already on the page.

Test plan

  • npm run lint, format:check, test:coverage (367 passed, thresholds met), build, size (within budget).
  • npm run test:e2e — 37 passed; sectionReveal.spec.js section list updated.
  • Screenshots at 1280px and 390px of About and Stack; checked the About column balance and that the three-column Stack keeps each item's name/description readable.

Made with Cursor

…, stop repeating the hero

The page covered the same four themes three sections running — About's
focus areas, the Skills cards, then the Stack — so a reader skimming for
what I do met one message three times. The Skills cards' one distinct asset
was their proof lines; those move into About's focus cards and the section
goes. Nav, scroll-spy, e2e section list and section numbers follow. Projects
takes the brand accent and the canvas/ink alternation is re-laid so removing
a section leaves no two neighbours on the same surface or hue.

Stack loses Azure, VPN, Windows and Network Administration: their
descriptions were dictionary definitions with nothing on the site behind
them, which undercut the closing card's claim that every entry has shipped
something. The two thinned categories merge into Systems & Networking and
the grid goes to three columns.

The hero's sentence was repeated on the experience card and in the footer;
both now say something the hero did not.

Co-authored-by: Cursor <cursoragent@cursor.com>
Copilot AI lite review requested due to automatic review settings September 3, 2026 06:08
@Aswinmcw
Aswinmcw requested review from a team and Aswin-coder as code owners September 3, 2026 06:08
@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Sep 3, 2026

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
aswin-portfolio fac3b63 Commit Preview URL

Branch Preview URL
Sep 03 2026, 06:09 AM

@Aswincloud-Bot Aswincloud-Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Auto-approved: @Aswinmcw is a member of @Aswincloud/admins.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Approval recommended

Changes are cohesive and the only issues found are minor documentation count mismatches that don’t affect runtime behavior.

Pull request overview

Refactors the About page flow by folding the former Skills “proof” content into About focus cards, removing the standalone Skills section, and tightening the Stack list to only include items that can be evidenced elsewhere on the site. It also reworks section numbering and surface/accent rhythm to keep adjacent sections visually distinct, and reduces repeated “AI accelerators” copy across the page.

Changes:

  • Remove SkillsSection and migrate its “proof lines” into AboutSection focus cards, including a pointer link to the Stack.
  • Trim/merge Stack categories (cut Azure/VPN/Windows/Network Admin; merge OS + Networking) and adjust layout/counting accordingly.
  • Renumber sections and update navigation/scroll tracking/tests to match the new section order and surfaces.
File summaries
File Description
src/index.css Updates inline docs to refer to AboutSection (not SkillsSection) for Tailwind literal-class constraints.
src/hooks/usePageTransitions.js Removes skills from the scroll-tracked section list.
src/data/sectionAccents.js Updates docs referencing AboutSection for literal class constraints.
src/data/projects.jsx Updates docs referencing AboutSection for literal class constraints.
src/data/experienceData.js Refreshes MulticoreWare description copy to be more specific (TT-Metal).
src/components/sections/TechnologiesSection.jsx Cuts unprovable items, merges categories, adjusts grid behavior, and updates section numbering/surface.
src/components/sections/SkillsSection.jsx Deletes the Skills section implementation.
src/components/sections/ProjectsSection.jsx Adjusts section surface/accent and renumbers to reflect removed Skills section.
src/components/sections/index.js Removes the SkillsSection export.
src/components/sections/Footer.jsx Switches footer surface to bg-ink and updates tagline copy.
src/components/sections/ContactSection.jsx Updates section surface and renumbers.
src/components/sections/AboutSection.jsx Absorbs Skills “proof” lines into focus cards, updates copy, and adds Stack pointer.
src/components/SectionHeader.jsx Minor doc tweak related to section sizing language.
src/components/Navigation.jsx Removes the Skills nav item.
src/components/icons/TechIcons.jsx Updates doc comment after removing Azure/Windows and other non-logo items.
src/components/background/TumblingCube.jsx Updates Tailwind literal-class docs to reference AboutSection focus cards.
src/components/tests/App.test.jsx Updates nav expectations to reflect Skills removal.
src/App.jsx Removes SkillsSection from the home page composition.
e2e/sectionReveal.spec.js Updates section list/count for reveal assertions after Skills removal.
Review details
  • Files reviewed: 19/19 changed files
  • Comments generated: 2
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread src/components/SectionHeader.jsx
Comment thread src/hooks/usePageTransitions.js
@Aswinmcw
Aswinmcw added this pull request to the merge queue Sep 3, 2026
Merged via the queue into main with commit 1411fb2 Sep 3, 2026
15 checks passed
@Aswinmcw
Aswinmcw deleted the refactor/fold-skills-into-about branch September 3, 2026 06:20
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.

3 participants